fix: add try catch on new URL that might fail (#1135)

graasp · Dec 9, 2024 · fe5ac27 · fe5ac27
1 parent b01618d
commit fe5ac27
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/src/items/LinkItem.tsx b/src/items/LinkItem.tsx
@@ -23,6 +23,14 @@ const StyledIFrame = styled('iframe')<{
   height: !isResizable ? height : '100%',
 }));
 
+function isURLExternal(url: string): boolean {
+  try {
+    return new URL(url).origin !== window.location.origin;
+  } catch {
+    return false;
+  }
+}
+
 type LinkItemProps = {
   /**
    * Id of the component used for testing
@@ -173,7 +181,7 @@ const LinkItem = ({
       return <Alert severity='error'>{errorMessage}</Alert>;
     }
 
-    const isExternal = new URL(url).origin !== window.location.origin;
+    const isExternal = isURLExternal(url);
     const linkCard = (
       <LinkCard
         id={id}