fix(nextjs): Rename nextjs.data.server ops (#5830)

`nextjs.data.server` -> `http.server` for transactions, and `nextjs.data.server` -> `function.nextjs` for spans.
getsentry · Sep 28, 2022 · fac1103 · fac1103
1 parent 186bcda
commit fac1103
Show file tree

Hide file tree

Showing 5 changed files with 8 additions and 8 deletions.
diff --git a/packages/nextjs/src/config/wrappers/wrapperUtils.ts b/packages/nextjs/src/config/wrappers/wrapperUtils.ts
@@ -88,7 +88,7 @@ export function callTracedServerSideDataFetcher<F extends (...args: any[]) => Pr
 
       const newTransaction = startTransaction(
         {
-          op: 'nextjs.data.server',
+          op: 'http.server',
           name: options.requestedRouteName,
           ...traceparentData,
           status: 'ok',
@@ -110,7 +110,7 @@ export function callTracedServerSideDataFetcher<F extends (...args: any[]) => Pr
     }
 
     const dataFetcherSpan = requestTransaction.startChild({
-      op: 'nextjs.data.server',
+      op: 'function.nextjs',
       description: `${options.dataFetchingMethodName} (${options.dataFetcherRouteName})`,
       status: 'ok',
     });
@@ -173,7 +173,7 @@ export async function callDataFetcherTraced<F extends (...args: any[]) => Promis
   // Capture the route, since pre-loading, revalidation, etc might mean that this span may happen during another
   // route's transaction
   const span = transaction.startChild({
-    op: 'nextjs.data.server',
+    op: 'function.nextjs',
     description: `${dataFetchingMethodName} (${parameterizedRoute})`,
     status: 'ok',
   });

diff --git a/packages/nextjs/test/config/wrappers.test.ts b/packages/nextjs/test/config/wrappers.test.ts
@@ -42,7 +42,7 @@ describe('data-fetching function wrappers', () => {
       expect(startTransactionSpy).toHaveBeenCalledWith(
         expect.objectContaining({
           name: '/tricks/[trickName]',
-          op: 'nextjs.data.server',
+          op: 'http.server',
           metadata: expect.objectContaining({ source: 'route' }),
         }),
         {
@@ -64,7 +64,7 @@ describe('data-fetching function wrappers', () => {
       expect(startTransactionSpy).toHaveBeenCalledWith(
         expect.objectContaining({
           name: '/tricks/[trickName]',
-          op: 'nextjs.data.server',
+          op: 'http.server',
           metadata: expect.objectContaining({ source: 'route' }),
         }),
         {

diff --git a/packages/nextjs/test/integration/test/server/errorServerSideProps.js b/packages/nextjs/test/integration/test/server/errorServerSideProps.js
@@ -32,7 +32,7 @@ module.exports = async ({ url: urlBase, argv }) => {
     {
       contexts: {
         trace: {
-          op: 'nextjs.data.server',
+          op: 'http.server',
           status: 'internal_error',
         },
       },

diff --git a/packages/nextjs/test/integration/test/server/tracingServerGetInitialProps.js b/packages/nextjs/test/integration/test/server/tracingServerGetInitialProps.js
@@ -10,7 +10,7 @@ module.exports = async ({ url: urlBase, argv }) => {
     {
       contexts: {
         trace: {
-          op: 'nextjs.data.server',
+          op: 'http.server',
           status: 'ok',
         },
       },

diff --git a/packages/nextjs/test/integration/test/server/tracingServerGetServerSideProps.js b/packages/nextjs/test/integration/test/server/tracingServerGetServerSideProps.js
@@ -10,7 +10,7 @@ module.exports = async ({ url: urlBase, argv }) => {
     {
       contexts: {
         trace: {
-          op: 'nextjs.data.server',
+          op: 'http.server',
           status: 'ok',
         },
       },