fix(tracing): Don't set method multiple times

Author: AbhiPrasad

packages/nextjs/test/integration/test/client/tracingFetch.test.ts

@@ -31,7 +31,7 @@ test('should correctly instrument `fetch` for performance tracing', async ({ pag
   expect(transaction[0].spans).toEqual(
     expect.arrayContaining([
       expect.objectContaining({
-        data: { method: 'GET', url: 'http://example.com', type: 'fetch' },
+        data: { 'http.method': 'GET', url: 'http://example.com', type: 'fetch' },
         description: 'GET http://example.com',
         op: 'http.client',
         parent_span_id: expect.any(String),

packages/tracing-internal/src/browser/request.ts

@@ -193,7 +193,7 @@ export function fetchCallback(
   if (currentSpan && activeTransaction) {
     const span = currentSpan.startChild({
       data: {
-        ...handlerData.fetchData,
+        url: handlerData.fetchData.url,
         type: 'fetch',
         'http.method': handlerData.fetchData.method,
       },