add backwards-compatibility-preserving wrappers to handlers

Author: lobsterkatie

packages/node/src/handlers.ts

@@ -239,3 +239,9 @@ export function errorHandler(options?: {
     next(error);
   };
 }
+
+// TODO (v8 / #5190): Remove this
+// eslint-disable-next-line deprecation/deprecation
+export type { ParseRequestOptions, ExpressRequest } from './requestDataDeprecated';
+// eslint-disable-next-line deprecation/deprecation
+export { parseRequest, extractRequestData } from './requestDataDeprecated';

packages/node/src/requestDataDeprecated.ts

@@ -0,0 +1,60 @@
+/**
+ * Deprecated functions which are slated for removal in v8. When the time comes, this entire file can be deleted.
+ *
+ * See XXXXXXX
+ */
+
+/* eslint-disable deprecation/deprecation */
+/* eslint-disable @typescript-eslint/no-explicit-any */
+import { Event, ExtractedNodeRequestData } from '@sentry/types';
+import {
+  addRequestDataToEvent,
+  AddRequestDataToEventOptions,
+  CrossPlatformRequest,
+  extractRequestData as _extractRequestData,
+} from '@sentry/utils';
+import * as cookie from 'cookie';
+import * as url from 'url';
+
+/**
+ * @deprecated `Handlers.ExpressRequest` is deprecated and will be removed in v8. Use `CrossPlatformRequest` instead.
+ */
+export type ExpressRequest = CrossPlatformRequest;
+
+/**
+ * Normalizes data from the request object, accounting for framework differences.
+ *
+ * @deprecated `Handlers.extractRequestData` is deprecated and will be removed in v8. Use `extractRequestData` instead.
+ *
+ * @param req The request object from which to extract data
+ * @param keys An optional array of keys to include in the normalized data.
+ * @returns An object containing normalized request data
+ */
+export function extractRequestData(req: { [key: string]: any }, keys?: string[]): ExtractedNodeRequestData {
+  return _extractRequestData(req, { include: keys, deps: { cookie, url } });
+}
+
+/**
+ * Options deciding what parts of the request to use when enhancing an event
+ *
+ * @deprecated `Handlers.ParseRequestOptions` is deprecated and will be removed in v8. Use
+ * `AddRequestDataToEventOptions` in `@sentry/utils` instead.
+ */
+export type ParseRequestOptions = AddRequestDataToEventOptions['include'] & {
+  serverName?: boolean;
+  version?: boolean;
+};
+
+/**
+ * Enriches passed event with request data.
+ *
+ * @deprecated `Handlers.parseRequest` is deprecated and will be removed in v8. Use `addRequestDataToEvent` instead.
+ *
+ * @param event Will be mutated and enriched with req data
+ * @param req Request object
+ * @param options object containing flags to enable functionality
+ * @hidden
+ */
+export function parseRequest(event: Event, req: ExpressRequest, options: ParseRequestOptions = {}): Event {
+  return addRequestDataToEvent(event, req, { include: options, deps: { cookie, url } });
+}