fix: avoid DOM Clobbering gadget in getRelativeUrlFromDocument (vitejs#18115)

jackfromeast · plchampigny · commit ad847e2fb586 · 2024-09-19T13:19:55.000-04:00
diff --git a/packages/vite/src/node/build.ts b/packages/vite/src/node/build.ts
@@ -1094,7 +1094,7 @@ const getRelativeUrlFromDocument = (relativePath: string, umd = false) =>
   getResolveUrl(
     `'${escapeId(relativePath)}', ${
       umd ? `typeof document === 'undefined' ? location.href : ` : ''
-    }document.currentScript && document.currentScript.src || document.baseURI`,
+    }document.currentScript && document.currentScript.tagName.toUpperCase() === 'SCRIPT' && document.currentScript.src || document.baseURI`,
   )
 
 const getFileUrlFromFullPath = (path: string) =>

Original file line number	Diff line number	Diff line change
`@@ -1094,7 +1094,7 @@ const getRelativeUrlFromDocument = (relativePath: string, umd = false) =>`
`1094`	`1094`	`getResolveUrl(`
`1095`	`1095`	`'${escapeId(relativePath)}', ${
`1096`	`1096`	umd ? `typeof document === 'undefined' ? location.href : ` : ''
`1097`		- }document.currentScript && document.currentScript.src \|\| document.baseURI`,
	`1097`	+ }document.currentScript && document.currentScript.tagName.toUpperCase() === 'SCRIPT' && document.currentScript.src \|\| document.baseURI`,
`1098`	`1098`	`)`
`1099`	`1099`
`1100`	`1100`	`const getFileUrlFromFullPath = (path: string) =>`