diff --git a/packages/utils/src/requestdata.ts b/packages/utils/src/requestdata.ts index f6a4129fac2a..807bc0541c2c 100644 --- a/packages/utils/src/requestdata.ts +++ b/packages/utils/src/requestdata.ts @@ -2,7 +2,6 @@ import type { Event, ExtractedNodeRequestData, PolymorphicRequest, - Transaction, TransactionSource, WebFetchHeaders, WebFetchRequest, @@ -24,17 +23,6 @@ const DEFAULT_INCLUDES = { const DEFAULT_REQUEST_INCLUDES = ['cookies', 'data', 'headers', 'method', 'query_string', 'url']; export const DEFAULT_USER_INCLUDES = ['id', 'username', 'email']; -type InjectedNodeDeps = { - cookie: { - parse: (cookieStr: string) => Record; - }; - url: { - parse: (urlStr: string) => { - query: string | null; - }; - }; -}; - /** * Options deciding what parts of the request to use when enhancing an event */ @@ -62,45 +50,6 @@ export type AddRequestDataToEventOptions = { export type TransactionNamingScheme = 'path' | 'methodPath' | 'handler'; -/** - * Sets parameterized route as transaction name e.g.: `GET /users/:id` - * Also adds more context data on the transaction from the request - */ -export function addRequestDataToTransaction( - transaction: Transaction | undefined, - req: PolymorphicRequest, - // TODO(v8): Remove this parameter in v8 - _deps?: InjectedNodeDeps, -): void { - if (!transaction) return; - - // TODO(v8): SEMANTIC_ATTRIBUTE_SENTRY_SOURCE is in core, align this once we merge utils & core - // eslint-disable-next-line deprecation/deprecation - if (!transaction.attributes['sentry.source'] || transaction.attributes['sentry.source'] === 'url') { - // Attempt to grab a parameterized route off of the request - const [name, source] = extractPathForTransaction(req, { path: true, method: true }); - transaction.updateName(name); - // TODO(v8): SEMANTIC_ATTRIBUTE_SENTRY_SOURCE is in core, align this once we merge utils & core - transaction.setAttribute('sentry.source', source); - } - transaction.setAttribute('url', req.originalUrl || req.url); - if (req.baseUrl) { - transaction.setAttribute('baseUrl', req.baseUrl); - } - - const query = extractQueryParams(req); - if (typeof query === 'string') { - transaction.setAttribute('query', query); - } else if (query) { - Object.keys(query).forEach(key => { - const val = query[key]; - if (typeof val === 'string' || typeof val === 'number') { - transaction.setAttribute(`query.${key}`, val); - } - }); - } -} - /** * Extracts a complete and parameterized path from the request object and uses it to construct transaction name. * If the parameterized transaction name cannot be extracted, we fall back to the raw URL.