add RequestData to default server integrations

Author: lobsterkatie

packages/nextjs/src/index.server.ts

@@ -1,6 +1,6 @@
 import { Carrier, getHubFromCarrier, getMainCarrier } from '@sentry/hub';
-import { RewriteFrames } from '@sentry/integrations';
-import { configureScope, getCurrentHub, init as nodeInit, Integrations } from '@sentry/node';
+import { RequestData, RewriteFrames } from '@sentry/integrations';
+import { addRequestDataToEvent, configureScope, getCurrentHub, init as nodeInit, Integrations } from '@sentry/node';
 import { hasTracingEnabled } from '@sentry/tracing';
 import { EventProcessor } from '@sentry/types';
 import { escapeStringForRegex, logger } from '@sentry/utils';
@@ -111,6 +111,16 @@ function addServerIntegrations(options: NextjsOptions): void {
   });
   integrations = addOrUpdateIntegration(defaultRewriteFramesIntegration, integrations);
 
+  const defaultRequestDataIntegration = new RequestData({ _addReqDataCallback: addRequestDataToEvent });
+  integrations = addOrUpdateIntegration(defaultRequestDataIntegration, integrations, {
+    // Specify the `@sentry/node` version of `addRequestDataToEvent`, so we get the injected dependencies
+    '_options._addReqDataCallback': addRequestDataToEvent,
+    // When the `transaction` option is set to true, it tries to extract a transaction name from the request object. We
+    // don't want this since we already have a high-quality transaction name with a parameterized route. Setting
+    // `transaction` to `true` will clobber that transaction name.
+    '_options.include.transaction': false,
+  });
+
   if (hasTracingEnabled(options)) {
     const defaultHttpTracingIntegration = new Integrations.Http({ tracing: true });
     integrations = addOrUpdateIntegration(defaultHttpTracingIntegration, integrations, {