#1621 fix: correct escaping of backslashes in URL wrapping

ruchira-net · ruchira-net · commit 0d0e829cb9d9 · 2024-11-22T16:09:38.000+13:00
diff --git a/src/runtime/getUrl.js b/src/runtime/getUrl.js
@@ -21,7 +21,7 @@ module.exports = (url, options) => {
   // Should url be wrapped?
   // See https://drafts.csswg.org/css-values-3/#urls
   if (/["'() \t\n]|(%20)/.test(url) || options.needQuotes) {
-    return `"${url.replace(/"/g, '\\"').replace(/\n/g, "\\n")}"`;
+    return `"${url.replace(/\\/g, '\\\\').replace(/"/g, '\\"').replace(/\n/g, "\\n')}"`;
   }
 
   return url;

Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,7 @@ module.exports = (url, options) => {`
`21`	`21`	`// Should url be wrapped?`
`22`	`22`	`// See https://drafts.csswg.org/css-values-3/#urls`
`23`	`23`	`if (/["'() \t\n]\|(%20)/.test(url) \|\| options.needQuotes) {`
`24`		- return `"${url.replace(/"/g, '\\"').replace(/\n/g, "\\n")}"`;
	`24`	+ return `"${url.replace(/\\/g, '\\\\').replace(/"/g, '\\"').replace(/\n/g, "\\n')}"`;
`25`	`25`	`}`
`26`	`26`
`27`	`27`	`return url;`