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

jackfromeast · patak-dev · commit e8127166979e · 2024-09-16T21:03:08.000+02:00
diff --git a/packages/vite/src/node/build.ts b/packages/vite/src/node/build.ts
@@ -1100,7 +1100,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
`@@ -1100,7 +1100,7 @@ const getRelativeUrlFromDocument = (relativePath: string, umd = false) =>`
`1100`	`1100`	`getResolveUrl(`
`1101`	`1101`	`'${escapeId(relativePath)}', ${
`1102`	`1102`	umd ? `typeof document === 'undefined' ? location.href : ` : ''
`1103`		- }document.currentScript && document.currentScript.src \|\| document.baseURI`,
	`1103`	+ }document.currentScript && document.currentScript.tagName.toUpperCase() === 'SCRIPT' && document.currentScript.src \|\| document.baseURI`,
`1104`	`1104`	`)`
`1105`	`1105`
`1106`	`1106`	`const getFileUrlFromFullPath = (path: string) =>`