Fix that we would have tried to capture an asset with a malformed url…

… with { origins: true }

packages/rrweb-snapshot/src/utils.ts

@@ -550,18 +550,17 @@ export function shouldIgnoreAsset(
   for (const origin of originsToIgnore) {
     if (url.startsWith(origin)) return true;
   }
-
+  let urlOrigin;
+  try {
+    urlOrigin = new URL(url).origin;
+  } catch (e) {
+    return true; // something went wrong, ignore!
+  }
   // Check the origins
   const captureOrigins = config.origins;
   if (typeof captureOrigins === 'boolean') {
     return !captureOrigins;
   } else if (Array.isArray(captureOrigins)) {
-    let urlOrigin;
-    try {
-      urlOrigin = new URL(url).origin;
-    } catch (e) {
-      return true; // something went wrong, ignore!
-    }
     return !captureOrigins.includes(urlOrigin);
   }
   return true; // no config, ignore!

packages/rrweb-snapshot/test/utils.test.ts

@@ -311,6 +311,9 @@ describe('utils', () => {
         shouldIgnoreAsset('http:', { origins: ['http://example.com'] }),
       ).toBe(true);
     });
+
+    it(`should ignore malformed url even with origins: true`, () => {
+      expect(shouldIgnoreAsset('http:', { origins: true })).toBe(true);
     });
   });