add tests

sorry

Author: Lms24

packages/sveltekit/src/client/load.ts

@@ -9,6 +9,7 @@ import {
   getSanitizedUrlString,
   objectify,
   parseFetchArgs,
+  parseUrl,
   stringMatchesSomePattern,
 } from '@sentry/utils';
 import type { LoadEvent } from '@sveltejs/kit';
@@ -107,12 +108,12 @@ function instrumentSvelteKitFetch(originalFetch: SvelteKitFetch): SvelteKitFetch
 
   const browserTracingIntegration =
     client.getIntegrationById && (client.getIntegrationById('BrowserTracing') as BrowserTracing | undefined);
-  const breadcrumbsIntegration = client.getIntegrationById('BreadCrumbs') as Breadcrumbs | undefined;
+  const breadcrumbsIntegration = client.getIntegrationById('Breadcrumbs') as Breadcrumbs | undefined;
 
   const browserTracingOptions = browserTracingIntegration && browserTracingIntegration.options;
 
   const shouldTraceFetch = browserTracingOptions && browserTracingOptions.traceFetch;
-  const shouldAddFetchBreadcrumbs = breadcrumbsIntegration && breadcrumbsIntegration.options.fetch;
+  const shouldAddFetchBreadcrumb = breadcrumbsIntegration && breadcrumbsIntegration.options.fetch;
 
   /* Identical check as in BrowserTracing, just that we also need to verify that BrowserTracing is actually installed */
   const shouldCreateSpan =
@@ -132,7 +133,8 @@ function instrumentSvelteKitFetch(originalFetch: SvelteKitFetch): SvelteKitFetch
     apply: (wrappingTarget, thisArg, args: Parameters<LoadEvent['fetch']>) => {
       const [input, init] = args;
       const { url: rawUrl, method } = parseFetchArgs(args);
-      const urlObject = new URL(rawUrl);
+      const urlObject = parseUrl(rawUrl);
+
       const requestData: SanitizedRequestData = {
         url: getSanitizedUrlString(urlObject),
         method,
@@ -156,6 +158,7 @@ function instrumentSvelteKitFetch(originalFetch: SvelteKitFetch): SvelteKitFetch
       // only attach headers if we should create a span
       if (attachHeaders && createSpan) {
         const dsc = activeTransaction.getDynamicSamplingContext();
+
         const headers = addTracingHeadersToFetchRequest(
           input as string | Request,
           dsc,
@@ -173,6 +176,8 @@ function instrumentSvelteKitFetch(originalFetch: SvelteKitFetch): SvelteKitFetch
 
       let fetchPromise: Promise<Response>;
 
+      const patchedFetchArgs = [input, patchedInit];
+
       if (createSpan) {
         fetchPromise = trace(
           {
@@ -181,18 +186,18 @@ function instrumentSvelteKitFetch(originalFetch: SvelteKitFetch): SvelteKitFetch
             data: requestData,
           },
           span => {
-            const promise: Promise<Response> = wrappingTarget.apply(thisArg, [input, patchedInit]);
+            const promise: Promise<Response> = wrappingTarget.apply(thisArg, patchedFetchArgs);
             if (span) {
               promise.then(res => span.setHttpStatus(res.status)).catch(_ => span.setStatus('internal_error'));
             }
             return promise;
           },
         );
       } else {
-        fetchPromise = wrappingTarget.apply(thisArg, [input, patchedInit]);
+        fetchPromise = wrappingTarget.apply(thisArg, patchedFetchArgs);
       }
 
-      if (shouldAddFetchBreadcrumbs) {
+      if (shouldAddFetchBreadcrumb) {
         addFetchBreadcrumb(fetchPromise, requestData, args);
       }