Fixed regression breaking network-based CSP injection.

hackademix · hackademix · commit 4d18298930fa · 2020-07-10T18:18:32.000+02:00
diff --git a/src/bg/ReportingCSP.js b/src/bg/ReportingCSP.js
@@ -7,7 +7,7 @@ function ReportingCSP(marker, reportURI = "") {
   
   return Object.assign(
     new CapsCSP(new NetCSP( 
-      reportURI ? `report-uri ${reportURI};` : marker
+      reportURI ? `report-uri ${reportURI}` : marker
     )),
     {
       reportURI,
diff --git a/src/lib/NetCSP.js b/src/lib/NetCSP.js
@@ -19,7 +19,7 @@ class NetCSP extends CSP {
   }
 
   build(...directives) {
-    return `${this.start}${super.build(...directives)}`;
+    return `${this.start};${super.build(...directives)}`;
   }
 
   cleanup(headers) {

Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@ function ReportingCSP(marker, reportURI = "") {`
`7`	`7`
`8`	`8`	`return Object.assign(`
`9`	`9`	`new CapsCSP(new NetCSP(`
`10`		- reportURI ? `report-uri ${reportURI};` : marker
	`10`	+ reportURI ? `report-uri ${reportURI}` : marker
`11`	`11`	`)),`
`12`	`12`	`{`
`13`	`13`	`reportURI,`
Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,7 @@ class NetCSP extends CSP {`
`19`	`19`	`}`
`20`	`20`
`21`	`21`	`build(...directives) {`
`22`		- return `${this.start}${super.build(...directives)}`;
	`22`	+ return `${this.start};${super.build(...directives)}`;
`23`	`23`	`}`
`24`	`24`
`25`	`25`	`cleanup(headers) {`