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

getsentry · Nov 28, 2023 · 28450ad · 28450ad
1 parent f97020f
commit 28450ad
Showing 1 changed file with 11 additions and 7 deletions.
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;
+  }
 }
 
 /**