ref(browser): Remove truncation when not needed (#18051)

Removes the remaining places where we truncate, although not needed.

part of #17389

Author: s1gr1d

packages/browser-utils/src/metrics/lcp.ts

@@ -77,8 +77,7 @@ export function _sendStandaloneLcpSpan(
     entry.element && (attributes['lcp.element'] = htmlTreeAsString(entry.element));
     entry.id && (attributes['lcp.id'] = entry.id);
 
-    // Trim URL to the first 200 characters.
-    entry.url && (attributes['lcp.url'] = entry.url.trim().slice(0, 200));
+    entry.url && (attributes['lcp.url'] = entry.url);
 
     // loadTime is the time of LCP that's related to receiving the LCP element response..
     entry.loadTime != null && (attributes['lcp.loadTime'] = entry.loadTime);

packages/browser/src/integrations/globalhandlers.ts

@@ -217,5 +217,5 @@ function getFilenameFromUrl(url: string | undefined): string | undefined {
     return `<data:${mimeType}${isBase64 ? ',base64' : ''}>`;
   }
 
-  return url.slice(0, 1024);
+  return url; // it's fine to not truncate it as it's not put in a regex (https://codeql.github.com/codeql-query-help/javascript/js-polynomial-redos)
 }