fix: useGif crashing renderer when no valid url was given

ZickZenni · Sep 18, 2024 · 71a981e · 71a981e
1 parent 70be4c7
commit 71a981e
Showing 1 changed file with 15 additions and 11 deletions.
diff --git a/src/renderer/hooks/useGif.ts b/src/renderer/hooks/useGif.ts
@@ -7,20 +7,24 @@ export default function useGif(url: string): TenorGif | null {
   const [gif, setGif] = useState<TenorGif | null>(null);
 
   useEffect(() => {
-    const uri = new URL(url);
+    try {
+      const uri = new URL(url);
 
-    if (allowedHosts.includes(uri.host.toLowerCase())) {
+      if (allowedHosts.includes(uri.host.toLowerCase())) {
+        setGif(null);
+        return;
+      }
+
+      window.electron.ipcRenderer
+        .invoke('tenor:fetch-gif', url)
+        .then((result: TenorFetchResult) => {
+          setGif(result.gif);
+          return true;
+        })
+        .catch((err) => console.error(err));
+    } catch {
       setGif(null);
-      return;
     }
-
-    window.electron.ipcRenderer
-      .invoke('tenor:fetch-gif', url)
-      .then((result: TenorFetchResult) => {
-        setGif(result.gif);
-        return true;
-      })
-      .catch((err) => console.error(err));
   }, [url]);
 
   return gif;