fix: parse percent-encoding url (#136)

Author: cap-Bernardito

package-lock.json

@@ -70,7 +70,7 @@
     "npm-run-all": "^4.1.5",
     "prettier": "^2.1.2",
     "standard-version": "^9.0.0",
-    "webpack": "^5.2.0"
+    "webpack": "^5.9.0"
   },
   "keywords": [
     "webpack"

package.json

@@ -87,8 +87,12 @@ function getSourceMappingURL(code) {
     }
   }
 
+  const sourceMappingURL = match ? match[1] || match[2] || '' : null;
+
   return {
-    sourceMappingURL: match ? match[1] || match[2] || '' : null,
+    sourceMappingURL: sourceMappingURL
+      ? decodeURI(sourceMappingURL)
+      : sourceMappingURL,
     replacementString: match ? match[0] : null,
   };
 }

src/utils.js

@@ -334,6 +334,16 @@ Object {
 
 exports[`source-map-loader should process inlined sources: warnings 1`] = `Array []`;
 
+exports[`source-map-loader should process percent-encoding path: css 1`] = `
+"with SourceMap
+// comment
+"
+`;
+
+exports[`source-map-loader should process percent-encoding path: errors 1`] = `Array []`;
+
+exports[`source-map-loader should process percent-encoding path: warnings 1`] = `Array []`;
+
 exports[`source-map-loader should process protocol-relative-url-path: css 1`] = `
 "// Some content
 // # sourceMappingURL=//sampledomain.com/external-source-map2.map

test/__snapshots__/loader.test.js.snap

@@ -0,0 +1,3 @@
+with SourceMap
+// #sourceMappingURL=%7Bpercent%7D.js.map
+// comment

test/fixtures/{percent}.js

@@ -51,6 +51,18 @@ describe('source-map-loader', () => {
     expect(getErrors(stats)).toMatchSnapshot('errors');
   });
 
+  it('should process percent-encoding path', async () => {
+    const testId = '{percent}.js';
+    const compiler = getCompiler(testId);
+    const stats = await compile(compiler);
+    const codeFromBundle = getCodeFromBundle(stats, compiler);
+
+    expect(codeFromBundle.map).toBeDefined();
+    expect(codeFromBundle.css).toMatchSnapshot('css');
+    expect(getWarnings(stats)).toMatchSnapshot('warnings');
+    expect(getErrors(stats)).toMatchSnapshot('errors');
+  });
+
   it('should process external SourceMaps', async () => {
     const testId = 'external-source-map.js';
     const compiler = getCompiler(testId);