From 858696626c6390f2971db1c5acd268e1afba4bc5 Mon Sep 17 00:00:00 2001
From: Katie Byers <lobsterkatie@gmail.com>
Date: Mon, 20 Jun 2022 12:57:00 -0700
Subject: [PATCH] add `ParseRequestOptions` back in as option for request
 handlers

---
 packages/node/src/handlers.ts               | 24 +++++++++++++++++----
 packages/serverless/src/gcpfunction/http.ts | 10 +++++++--
 2 files changed, 28 insertions(+), 6 deletions(-)

diff --git a/packages/node/src/handlers.ts b/packages/node/src/handlers.ts
index 81496ce2a66f..95158d28390e 100644
--- a/packages/node/src/handlers.ts
+++ b/packages/node/src/handlers.ts
@@ -14,6 +14,9 @@ import * as domain from 'domain';
 import * as http from 'http';
 
 import { NodeClient } from './client';
+// TODO (v8 / XXX) Remove these imports
+import type { ParseRequestOptions } from './requestDataDeprecated';
+import { parseRequest } from './requestDataDeprecated';
 import { addRequestDataToEvent, extractRequestData, flush, isAutoSessionTrackingEnabled } from './sdk';
 
 /**
@@ -72,9 +75,12 @@ export function tracingHandler(): (
   };
 }
 
-export type RequestHandlerOptions = AddRequestDataToEventOptions & {
-  flushTimeout?: number;
-};
+export type RequestHandlerOptions =
+  // TODO (v8 / XXX) Remove ParseRequestOptions type and eslint override
+  // eslint-disable-next-line deprecation/deprecation
+  | (ParseRequestOptions | AddRequestDataToEventOptions) & {
+      flushTimeout?: number;
+    };
 
 /**
  * Express compatible request handler.
@@ -96,11 +102,21 @@ export function requestHandler(
       scope.setSession();
     }
   }
+
   return function sentryRequestMiddleware(
     req: http.IncomingMessage,
     res: http.ServerResponse,
     next: (error?: any) => void,
   ): void {
+    // TODO (v8 / XXX) Remove this shim and just use `addRequestDataToEvent`
+    let backwardsCompatibleEventProcessor: (event: Event) => Event;
+    if (options && 'include' in options) {
+      backwardsCompatibleEventProcessor = (event: Event) => addRequestDataToEvent(event, req, options);
+    } else {
+      // eslint-disable-next-line deprecation/deprecation
+      backwardsCompatibleEventProcessor = (event: Event) => parseRequest(event, req, options as ParseRequestOptions);
+    }
+
     if (options && options.flushTimeout && options.flushTimeout > 0) {
       // eslint-disable-next-line @typescript-eslint/unbound-method
       const _end = res.end;
@@ -124,7 +140,7 @@ export function requestHandler(
       const currentHub = getCurrentHub();
 
       currentHub.configureScope(scope => {
-        scope.addEventProcessor((event: Event) => addRequestDataToEvent(event, req, options));
+        scope.addEventProcessor(backwardsCompatibleEventProcessor);
         const client = currentHub.getClient<NodeClient>();
         if (isAutoSessionTrackingEnabled(client)) {
           const scope = currentHub.getScope();
diff --git a/packages/serverless/src/gcpfunction/http.ts b/packages/serverless/src/gcpfunction/http.ts
index 2666cefc4e87..84551170c4de 100644
--- a/packages/serverless/src/gcpfunction/http.ts
+++ b/packages/serverless/src/gcpfunction/http.ts
@@ -13,11 +13,16 @@ import { domainify, getActiveDomain, proxyFunction } from './../utils';
 import { HttpFunction, WrapperOptions } from './general';
 
 // TODO (v8 / #5257): Remove this whole old/new business and just use the new stuff
+type ParseRequestOptions = AddRequestDataToEventOptions['include'] & {
+  serverName?: boolean;
+  version?: boolean;
+};
+
 interface OldHttpFunctionWrapperOptions extends WrapperOptions {
   /**
    * @deprecated Use `addRequestDataToEventOptions` instead.
    */
-  parseRequestOptions: AddRequestDataToEventOptions;
+  parseRequestOptions: ParseRequestOptions;
 }
 interface NewHttpFunctionWrapperOptions extends WrapperOptions {
   addRequestDataToEventOptions: AddRequestDataToEventOptions;
@@ -58,7 +63,8 @@ function _wrapHttpFunction(fn: HttpFunction, wrapOptions: Partial<HttpFunctionWr
 
   const options: HttpFunctionWrapperOptions = {
     flushTimeout: 2000,
-    addRequestDataToEventOptions: parseRequestOptions ? parseRequestOptions : {},
+    // TODO (v8 / xxx): Remove this line, since `addRequestDataToEventOptions` will be included in the spread of `wrapOptions`
+    addRequestDataToEventOptions: parseRequestOptions ? { include: parseRequestOptions } : {},
     ...wrapOptions,
   };
   return (req, res) => {