remove URLSearchParams

Author: gingerbenw

packages/plugin-network-instrumentation/lib/extract-domain.js

@@ -5,8 +5,19 @@
    */
 module.exports = function (url) {
   try {
-    const urlObj = new URL(url)
-    return urlObj.host
+    const isAbsolute = /^https?:\/\//i.test(url)
+    if (!isAbsolute) {
+      return 'unknown'
+    }
+
+    const urlWithoutProtocol = url.replace(/^https?:\/\//i, '')
+
+    const firstSlashIndex = urlWithoutProtocol.indexOf('/')
+    if (firstSlashIndex !== -1) {
+      return urlWithoutProtocol.substring(0, firstSlashIndex)
+    }
+
+    return urlWithoutProtocol
   } catch (e) {
     return 'unknown'
   }

packages/plugin-network-instrumentation/lib/parse-query-params.js

@@ -5,12 +5,24 @@
    */
 module.exports = function (url) {
   try {
-    const urlObj = new URL(url)
-    const params = {}
-    urlObj.searchParams.forEach((value, key) => {
-      params[key] = value
+    const queryStart = url.indexOf('?')
+    const hashStart = url.indexOf('#')
+
+    let queryString = ''
+    if (queryStart !== -1) {
+      const queryEnd = hashStart !== -1 && hashStart > queryStart ? hashStart : url.length
+      queryString = url.substring(queryStart + 1, queryEnd)
+    }
+
+    // convert query string to object without using UrlSearchParams
+    const queryStringObject = {}
+    const pairs = queryString.split('&').filter(pair => pair.length > 0)
+    pairs.forEach(pair => {
+      const [key, value] = pair.split('=')
+      queryStringObject[decodeURIComponent(key)] = decodeURIComponent(value || '')
     })
-    return params
+
+    return queryStringObject
   } catch (e) {
     return {}
   }

packages/plugin-network-instrumentation/lib/redact-query-parameters.js

@@ -1,55 +1,23 @@
+const parseQueryParams = require('./parse-query-params')
 const redactValues = require('./redact-values')
 
-function isAbsoluteURL (url) {
-  try {
-    // eslint-disable-next-line no-new
-    new URL(url)
-    return true
-  } catch (e) {
-    return false
-  }
-}
-
 module.exports = function (url, redactedKeys) {
-  const isAbsolute = isAbsoluteURL(url)
-  const base = isAbsolute ? undefined : 'http://localhost'
-
-  // Parse the URL - use a base only for relative URLs
-  const urlObj = new URL(url, base)
+  const paramsObject = parseQueryParams(url)
+  const redactedParams = redactValues(paramsObject, redactedKeys)
+  const redactedQueryString = Object.entries(redactedParams).map(([key, value]) => `${key}=${value}`).join('&')
 
-  // Extract query string manually from the original URL
   const queryStart = url.indexOf('?')
   const hashStart = url.indexOf('#')
-
-  let queryString = ''
-  if (queryStart !== -1) {
-    const queryEnd = hashStart !== -1 && hashStart > queryStart ? hashStart : url.length
-    queryString = url.substring(queryStart + 1, queryEnd)
-  }
-
-  // Convert URLSearchParams to object without using Object.fromEntries()
-  const params = new URLSearchParams(queryString)
-  const paramsObject = {}
-  params.forEach((value, key) => {
-    paramsObject[key] = value
-  })
-
-  const redactedParams = redactValues(paramsObject, redactedKeys)
-  const redactedQueryString = new URLSearchParams(redactedParams).toString()
+  const hash = hashStart !== -1 ? url.substring(hashStart) : ''
+  let result = queryStart !== -1 ? url.substring(0, queryStart) : url
 
   // Build the result URL manually
-  let result = urlObj.pathname
-  if (redactedQueryString) {
+  if (redactedQueryString && redactedQueryString.length > 0) {
     result += '?' + redactedQueryString
   }
-  if (urlObj.hash) {
-    result += urlObj.hash
-  }
-
-  // Return appropriate format based on original URL type
-  if (isAbsolute) {
-    return decodeURI(urlObj.origin + result)
+  if (hash) {
+    result += hash
   }
 
-  return decodeURI(result)
+  return result
 }