feat(core): Add parseStringToURL method

Author: AbhiPrasad

packages/core/src/fetch.ts

@@ -5,7 +5,7 @@ import { SentryNonRecordingSpan } from './tracing/sentryNonRecordingSpan';
 import type { FetchBreadcrumbHint, HandlerDataFetch, Span, SpanOrigin } from './types-hoist';
 import { SENTRY_BAGGAGE_KEY_PREFIX } from './utils-hoist/baggage';
 import { isInstanceOf } from './utils-hoist/is';
-import { parseUrl, stripUrlQueryAndFragment } from './utils-hoist/url';
+import { parseStringToURL, stripUrlQueryAndFragment } from './utils-hoist/url';
 import { hasSpansEnabled } from './utils/hasSpansEnabled';
 import { getActiveSpan } from './utils/spanUtils';
 import { getTraceData } from './utils/traceData';
@@ -53,8 +53,7 @@ export function instrumentFetchRequest(
     return undefined;
   }
 
-  const fullUrl = getFullURL(url);
-  const parsedUrl = fullUrl ? parseUrl(fullUrl) : parseUrl(url);
+  const parsedUrl = parseStringToURL(url);
 
   const hasParent = !!getActiveSpan();
 
@@ -66,12 +65,12 @@ export function instrumentFetchRequest(
             url,
             type: 'fetch',
             'http.method': method,
-            'http.url': fullUrl,
-            'server.address': parsedUrl?.host,
+            'http.url': parsedUrl?.href || url,
             [SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN]: spanOrigin,
             [SEMANTIC_ATTRIBUTE_SENTRY_OP]: 'http.client',
-            ...(parsedUrl?.search && { 'http.query': parsedUrl?.search }),
-            ...(parsedUrl?.hash && { 'http.fragment': parsedUrl?.hash }),
+            ...(parsedUrl?.host && { 'server.address': parsedUrl.host }),
+            ...(parsedUrl?.search && { 'http.query': parsedUrl.search }),
+            ...(parsedUrl?.hash && { 'http.fragment': parsedUrl.hash }),
           },
         })
       : new SentryNonRecordingSpan();
@@ -215,15 +214,6 @@ function _addTracingHeadersToFetchRequest(
   }
 }
 
-function getFullURL(url: string): string | undefined {
-  try {
-    const parsed = new URL(url);
-    return parsed.href;
-  } catch {
-    return undefined;
-  }
-}
-
 function endSpan(span: Span, handlerData: HandlerDataFetch): void {
   if (handlerData.response) {
     setHttpStatus(span, handlerData.response.status);

packages/core/src/utils-hoist/url.ts

@@ -7,6 +7,33 @@ type PartialURL = {
   hash?: string;
 };
 
+interface URLwithCanParse extends URL {
+  canParse: (url: string) => boolean;
+}
+
+/**
+ * Parses string to a URL object
+ *
+ * @param url - The URL to parse
+ * @returns The parsed URL object or undefined if the URL is invalid
+ */
+export function parseStringToURL(url: string): URL | undefined {
+  try {
+    // Node 20+, Chrome 120+, Firefox 115+, Safari 17+
+    if ('canParse' in URL) {
+      // Use `canParse` to short-circuit the URL constructor if it's not a valid URL
+      // This is faster than trying to construct the URL and catching the error
+      return (URL as unknown as URLwithCanParse).canParse(url) ? new URL(url) : undefined;
+    } else {
+      return new URL(url);
+    }
+  } catch {
+    // empty body
+  }
+
+  return undefined;
+}
+
 /**
  * Parses string form of URL into an object
  * // borrowed from https://tools.ietf.org/html/rfc3986#appendix-B

packages/core/test/utils-hoist/url.test.ts

@@ -1,5 +1,5 @@
 import { describe, expect, it } from 'vitest';
-import { getSanitizedUrlString, parseUrl, stripUrlQueryAndFragment } from '../../src/utils-hoist/url';
+import { getSanitizedUrlString, parseStringToURL, parseUrl, stripUrlQueryAndFragment } from '../../src/utils-hoist/url';
 
 describe('stripQueryStringAndFragment', () => {
   const urlString = 'http://dogs.are.great:1231/yay/';
@@ -196,3 +196,20 @@ describe('parseUrl', () => {
     expect(parseUrl(url)).toEqual(expected);
   });
 });
+
+describe('parseStringToURL', () => {
+  it('returns undefined for invalid URLs', () => {
+    expect(parseStringToURL('invalid-url')).toBeUndefined();
+  });
+
+  it('returns a URL object for valid URLs', () => {
+    expect(parseStringToURL('https://somedomain.com')).toBeInstanceOf(URL);
+  });
+
+  it('does not throw an error if URl.canParse is not defined', () => {
+    const canParse = (URL as any).canParse;
+    delete (URL as any).canParse;
+    expect(parseStringToURL('https://somedomain.com')).toBeInstanceOf(URL);
+    (URL as any).canParse = canParse;
+  });
+});