fix(posthtml#220): should not minify json with SRI

SukkaW · Jan 8, 2023 · 447079d · 447079d
1 parent 9681eaa
commit 447079d
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 0 deletions.
diff --git a/lib/modules/minifyJson.es6 b/lib/modules/minifyJson.es6
@@ -2,6 +2,11 @@ const rNodeAttrsTypeJson = /(\/|\+)json/;
 
 export function onContent() {
     return (content, node) => {
+        // Skip SRI
+        if (node.attrs && 'integrity' in node.attrs) {
+            return content;
+        }
+
         if (node.attrs && node.attrs.type && rNodeAttrsTypeJson.test(node.attrs.type)) {
             try {
                 // cast minified JSON to an array

diff --git a/test/modules/minifyJson.js b/test/modules/minifyJson.js
@@ -26,6 +26,25 @@ describe('minifyJson', () => {
         );
     });
 
+    it('should skip JSON inside <script> tags with SRI', () => {
+        const fixtures = `<script type="application/json" integrity="example">
+                {
+                    "test": 5
+                }
+             </script>
+             <script type="application/ld+json" integrity="example">
+                {
+                    "test": 6
+                }
+             </script>`;
+
+        return init(
+            fixtures,
+            fixtures,
+            options
+        );
+    });
+
     it('should skip <script> tags with non-JSON mime type', () => {
         return init(
             '<script>{"test": 5}</script>',