fix: ignore blob:// and data:// url (#23)

GopeedLab · Apr 19, 2024 · 4430079 · 4430079
1 parent beed726
commit 4430079
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/background/index.ts b/background/index.ts
@@ -83,6 +83,9 @@ export async function checkServer(server: Server): Promise<CheckResult> {
 const downloadEvent = chrome.downloads.onDeterminingFilename || chrome.downloads.onCreated
 
 downloadEvent.addListener(async function (item) {
+  const finalUrl = item.finalUrl || item.url
+  if (finalUrl.startsWith("blob:") || finalUrl.startsWith("data:")) return
+
   const server = await getSelectedServer()
   if (!server) return
 
@@ -94,7 +97,7 @@ downloadEvent.addListener(async function (item) {
   const asset = <Asset>{
     filename: path.basename(item.filename.replaceAll("\\", "/")),
     filesize: item.fileSize,
-    finalUrl: item.finalUrl || item.url
+    finalUrl
   }
   chrome.windows.getCurrent((currentWindow) => {
     const width = 480