fix(sandbox): fix blob file transform url (#672)

web-infra-dev · May 28, 2024 · f0c5cf1 · f0c5cf1
1 parent 31649b3
commit f0c5cf1
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 3 deletions.
diff --git a/packages/browser-vm/src/dynamicNode/processor.ts b/packages/browser-vm/src/dynamicNode/processor.ts
@@ -52,8 +52,16 @@ export class DynamicNodeProcessor {
       const src = el.getAttribute('src');
       const href = el.getAttribute('href');
       if (this.sandbox.options.fixStaticResourceBaseUrl) {
-        src && (el.src = transformUrl(baseUrl, src));
-        href && (el.href = transformUrl(baseUrl, href));
+        const transformUrlSrc = src && transformUrl(baseUrl, src);
+        const transformUrlHref = href && transformUrl(baseUrl, href);
+        // Consistent values do not need to be overwritten
+        if (transformUrlSrc !== src) {
+          el.src = transformUrlSrc;
+        }
+
+        if (transformUrlHref !== href) {
+          el.href = transformUrlHref;
+        }
       }
 
       const url = el.src || el.href;

diff --git a/packages/utils/src/utils.ts b/packages/utils/src/utils.ts
@@ -371,7 +371,11 @@ export function isAbsolute(url: string) {
 }
 
 export function transformUrl(resolvePath: string, curPath: string) {
-  if (curPath.startsWith('http') || curPath.startsWith('//')) {
+  if (
+    curPath.startsWith('http') ||
+    curPath.startsWith('//') ||
+    curPath.startsWith('blob:')
+  ) {
     return curPath;
   }
   const baseUrl = new URL(resolvePath, location.href);