fix(mc-html-template): permission policy to be joined with comma (#2170)

commercetools · Apr 29, 2021 · 4d4b1f9 · 4d4b1f9 · vercel · Apr 29, 2021
1 parent 75efb3e
commit 4d4b1f9
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/.changeset/odd-dodos-tickle.md b/.changeset/odd-dodos-tickle.md
@@ -0,0 +1,5 @@
+---
+"@commercetools-frontend/mc-html-template": patch
+---
+
+Fix multiple permissions policies to be joined with comma not semicolon.
diff --git a/packages/mc-html-template/src/process-headers.js b/packages/mc-html-template/src/process-headers.js
@@ -24,7 +24,7 @@ const mergeCspDirectives = (...csps) =>
       ),
     {}
   );
-const toHeaderString = (directives = {}) =>
+const toHeaderString = (directives = {}, seperator = '; ') =>
   Object.entries(directives)
     .map(
       ([directive, value]) =>
@@ -126,7 +126,8 @@ const processHeaders = (applicationConfig) => {
     }),
     ...(applicationConfig.headers.permissionsPolicies && {
       'Permissions-Policy': toHeaderString(
-        applicationConfig.headers.permissionsPolicies
+        applicationConfig.headers.permissionsPolicies,
+        ', '
       ),
     }),
   };