fix(utils): Try catch new URL when extracting query params (getsentry#9675)

AbhiPrasad · web-flow · commit 28450adc81fa · 2023-11-28T16:34:01.000Z
diff --git a/packages/utils/src/requestdata.ts b/packages/utils/src/requestdata.ts
@@ -357,13 +357,17 @@ function extractQueryParams(
     originalUrl = `http://dogs.are.great${originalUrl}`;
   }
 
-  return (
-    req.query ||
-    (typeof URL !== undefined && new URL(originalUrl).search.replace('?', '')) ||
-    // In Node 8, `URL` isn't in the global scope, so we have to use the built-in module from Node
-    (deps && deps.url && deps.url.parse(originalUrl).query) ||
-    undefined
-  );
+  try {
+    return (
+      req.query ||
+      (typeof URL !== undefined && new URL(originalUrl).search.slice(1)) ||
+      // In Node 8, `URL` isn't in the global scope, so we have to use the built-in module from Node
+      (deps && deps.url && deps.url.parse(originalUrl).query) ||
+      undefined
+    );
+  } catch {
+    return undefined;
+  }
 }
 
 /**
