From f76d40bec4b919c877a1e370aecbb471d33f2a55 Mon Sep 17 00:00:00 2001
From: wukko <me@wukko.me>
Date: Tue, 11 Feb 2025 15:29:23 +0600
Subject: [PATCH] web/PickerItem: make sure the item url is valid

---
 web/src/components/dialog/PickerItem.svelte | 22 +++++++++++++++------
 1 file changed, 16 insertions(+), 6 deletions(-)

diff --git a/web/src/components/dialog/PickerItem.svelte b/web/src/components/dialog/PickerItem.svelte
index 4a515a6e..04e301ec 100644
--- a/web/src/components/dialog/PickerItem.svelte
+++ b/web/src/components/dialog/PickerItem.svelte
@@ -14,18 +14,28 @@
     export let number: number;
 
     let imageLoaded = false;
-    const isTunnel = new URL(item.url).pathname === "/tunnel";
+
+    let validUrl = false;
+    try {
+        new URL(item.url);
+        validUrl = true;
+    } catch {}
+
+    const isTunnel = validUrl && new URL(item.url).pathname === "/tunnel";
 
     $: itemType = item.type ?? "photo";
 </script>
 
 <button
     class="picker-item"
-    on:click={() =>
-        downloadFile({
-            url: item.url,
-            urlType: isTunnel ? "tunnel" : "redirect",
-        })}
+    on:click={() => {
+        if (validUrl) {
+            downloadFile({
+                url: item.url,
+                urlType: isTunnel ? "tunnel" : "redirect",
+            });
+        }
+    }}
 >
     <div class="picker-type">
         {#if itemType === "video"}