Feature: escape deps url on CSSPackager

parcel-bundler · Mar 20, 2022 · 483258c · 483258c
1 parent 96c847b
commit 483258c
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/packages/core/utils/src/replaceBundleReferences.js b/packages/core/utils/src/replaceBundleReferences.js
@@ -191,6 +191,9 @@ export function getURLReplacement({
 
   let placeholder = dependency.meta?.placeholder ?? dependency.id;
   invariant(typeof placeholder === 'string');
+
+  to = to.replace(/"/g, '\\"');
+
   return {
     from: placeholder,
     to,

diff --git a/packages/transformers/css/src/CSSTransformer.js b/packages/transformers/css/src/CSSTransformer.js
@@ -83,12 +83,15 @@ export default (new Transformer({
             symbols: new Map([['*', {local: '*', isWeak: true, loc}]]),
           });
         } else if (dep.type === 'url') {
-          asset.addURLDependency(dep.url, {
-            loc,
-            meta: {
-              placeholder: dep.placeholder,
+          asset.addURLDependency(
+            dep.url.replace(/\\"/g, '"').replace(/\\'/g, "'"),
+            {
+              loc,
+              meta: {
+                placeholder: dep.placeholder,
+              },
             },
-          });
+          );
         }
       }
     }