s/CrossPlatformRequest/PolymorphicRequest

Author: lobsterkatie

packages/node/src/index.ts

@@ -1,7 +1,7 @@
 export type {
   Breadcrumb,
   BreadcrumbHint,
-  CrossPlatformRequest,
+  PolymorphicRequest,
   Request,
   SdkInfo,
   Event,

packages/node/src/requestDataDeprecated.ts

@@ -6,7 +6,7 @@
 
 /* eslint-disable deprecation/deprecation */
 /* eslint-disable @typescript-eslint/no-explicit-any */
-import { CrossPlatformRequest, Event, ExtractedNodeRequestData } from '@sentry/types';
+import { Event, ExtractedNodeRequestData, PolymorphicRequest } from '@sentry/types';
 import {
   addRequestDataToEvent,
   AddRequestDataToEventOptions,
@@ -16,9 +16,9 @@ import * as cookie from 'cookie';
 import * as url from 'url';
 
 /**
- * @deprecated `Handlers.ExpressRequest` is deprecated and will be removed in v8. Use `CrossPlatformRequest` instead.
+ * @deprecated `Handlers.ExpressRequest` is deprecated and will be removed in v8. Use `PolymorphicRequest` instead.
  */
-export type ExpressRequest = CrossPlatformRequest;
+export type ExpressRequest = PolymorphicRequest;
 
 /**
  * Normalizes data from the request object, accounting for framework differences.

packages/node/src/sdk.ts

@@ -1,7 +1,7 @@
 /* eslint-disable max-lines */
 import { getCurrentHub, getIntegrationsToSetup, initAndBind, Integrations as CoreIntegrations } from '@sentry/core';
 import { getMainCarrier, setHubOnCarrier } from '@sentry/hub';
-import { CrossPlatformRequest, Event, ExtractedNodeRequestData, SessionStatus, StackParser } from '@sentry/types';
+import { Event, ExtractedNodeRequestData, PolymorphicRequest, SessionStatus, StackParser } from '@sentry/types';
 import {
   addRequestDataToEvent as _addRequestDataToEvent,
   AddRequestDataToEventOptions,
@@ -293,7 +293,7 @@ function startSessionTracking(): void {
  */
 export function addRequestDataToEvent(
   event: Event,
-  req: CrossPlatformRequest,
+  req: PolymorphicRequest,
   options?: Omit<AddRequestDataToEventOptions, 'deps'>,
 ): Event {
   return _addRequestDataToEvent(event, req, {
@@ -317,7 +317,7 @@ export function addRequestDataToEvent(
  * @returns An object containing normalized request data
  */
 export function extractRequestData(
-  req: CrossPlatformRequest,
+  req: PolymorphicRequest,
   options?: {
     include?: string[];
   },

packages/tracing/src/integrations/node/express.ts

@@ -1,5 +1,5 @@
 /* eslint-disable max-lines */
-import { CrossPlatformRequest, Integration, Transaction } from '@sentry/types';
+import { Integration, PolymorphicRequest, Transaction } from '@sentry/types';
 import { extractPathForTransaction, getNumberOfUrlSegments, isRegExp, logger } from '@sentry/utils';
 
 type Method =
@@ -33,8 +33,8 @@ type Router = {
   [method in Method]: (...args: any) => any; // eslint-disable-line @typescript-eslint/no-explicit-any
 };
 
-/* Extend the CrossPlatformRequest type with a patched parameter to build a reconstructed route */
-type PatchedRequest = CrossPlatformRequest & { _reconstructedRoute?: string };
+/* Extend the PolymorphicRequest type with a patched parameter to build a reconstructed route */
+type PatchedRequest = PolymorphicRequest & { _reconstructedRoute?: string };
 
 /* Types used for patching the express router prototype */
 type ExpressRouter = Router & {

packages/types/src/index.ts

@@ -39,7 +39,7 @@ export type { Mechanism } from './mechanism';
 export type { ExtractedNodeRequestData, HttpHeaderValue, Primitive, WorkerLocation } from './misc';
 export type { ClientOptions, Options } from './options';
 export type { Package } from './package';
-export type { PolymorphicEvent, CrossPlatformRequest } from './polymorphics';
+export type { PolymorphicEvent, PolymorphicRequest } from './polymorphics';
 export type { QueryParams, Request } from './request';
 export type { Runtime } from './runtime';
 export type { CaptureContext, Scope, ScopeContext } from './scope';

packages/types/src/polymorphics.ts

@@ -13,7 +13,7 @@ export interface PolymorphicEvent {
 }
 
 /** A `Request` type compatible with Node, Express, browser, etc., because everything is optional */
-export type CrossPlatformRequest = BaseRequest &
+export type PolymorphicRequest = BaseRequest &
   BrowserRequest &
   NodeRequest &
   ExpressRequest &

packages/types/src/transaction.ts

@@ -1,7 +1,7 @@
 import { Baggage } from './baggage';
 import { MeasurementUnit } from './measurement';
 import { ExtractedNodeRequestData, Primitive, WorkerLocation } from './misc';
-import { CrossPlatformRequest } from './polymorphics';
+import { PolymorphicRequest } from './polymorphics';
 import { Span, SpanContext } from './span';
 /**
  * Interface holding Transaction-specific properties
@@ -143,6 +143,9 @@ export interface TransactionMetadata {
   /** The baggage object of a transaction's baggage header, used for dynamic sampling  */
   baggage?: Baggage;
 
+  /** For transactions tracing server-side request handling, the request being tracked. */
+  request?: PolymorphicRequest;
+
   /** For transactions tracing server-side request handling, the path of the request being tracked. */
   requestPath?: string;
 

packages/utils/src/requestdata.ts

@@ -12,7 +12,7 @@
 /* eslint-disable max-lines */
 /* eslint-disable @typescript-eslint/no-explicit-any */
 
-import { CrossPlatformRequest, Event, ExtractedNodeRequestData, Transaction, TransactionSource } from '@sentry/types';
+import { Event, ExtractedNodeRequestData, PolymorphicRequest, Transaction, TransactionSource } from '@sentry/types';
 
 import { isPlainObject, isString } from './is';
 import { normalize } from './normalize';
@@ -44,7 +44,7 @@ type InjectedNodeDeps = {
  */
 export function addRequestDataToTransaction(
   transaction: Transaction | undefined,
-  req: CrossPlatformRequest,
+  req: PolymorphicRequest,
   deps?: InjectedNodeDeps,
 ): void {
   if (!transaction) return;
@@ -74,7 +74,7 @@ export function addRequestDataToTransaction(
  * @returns A tuple of the fully constructed transaction name [0] and its source [1] (can be either 'route' or 'url')
  */
 export function extractPathForTransaction(
-  req: CrossPlatformRequest,
+  req: PolymorphicRequest,
   options: { path?: boolean; method?: boolean; customRoute?: string } = {},
 ): [string, TransactionSource] {
   const method = req.method && req.method.toUpperCase();
@@ -110,7 +110,7 @@ export function extractPathForTransaction(
 type TransactionNamingScheme = 'path' | 'methodPath' | 'handler';
 
 /** JSDoc */
-function extractTransaction(req: CrossPlatformRequest, type: boolean | TransactionNamingScheme): string {
+function extractTransaction(req: PolymorphicRequest, type: boolean | TransactionNamingScheme): string {
   switch (type) {
     case 'path': {
       return extractPathForTransaction(req, { path: true })[0];
@@ -154,7 +154,7 @@ function extractUserData(
  * @returns An object containing normalized request data
  */
 export function extractRequestData(
-  req: CrossPlatformRequest,
+  req: PolymorphicRequest,
   options?: {
     include?: string[];
     deps?: InjectedNodeDeps;
@@ -282,7 +282,7 @@ export interface AddRequestDataToEventOptions {
  */
 export function addRequestDataToEvent(
   event: Event,
-  req: CrossPlatformRequest,
+  req: PolymorphicRequest,
   options?: AddRequestDataToEventOptions,
 ): Event {
   const include = {
@@ -335,7 +335,7 @@ export function addRequestDataToEvent(
 }
 
 function extractQueryParams(
-  req: CrossPlatformRequest,
+  req: PolymorphicRequest,
   deps?: InjectedNodeDeps,
 ): string | Record<string, unknown> | undefined {
   // url (including path and query string):