From dffb0d5a70fe6791777a59261250e6f308f9d09d Mon Sep 17 00:00:00 2001 From: Abhijeet Prasad Date: Tue, 5 Jul 2022 15:06:04 -0400 Subject: [PATCH] fix(remix): Strip query params from transaction names --- packages/remix/src/utils/instrumentServer.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/remix/src/utils/instrumentServer.ts b/packages/remix/src/utils/instrumentServer.ts index 486ab06b149d..f11eb2186ab8 100644 --- a/packages/remix/src/utils/instrumentServer.ts +++ b/packages/remix/src/utils/instrumentServer.ts @@ -1,6 +1,6 @@ import { captureException, configureScope, getCurrentHub, startTransaction } from '@sentry/node'; import { getActiveTransaction } from '@sentry/tracing'; -import { addExceptionMechanism, fill, loadModule, logger } from '@sentry/utils'; +import { addExceptionMechanism, fill, loadModule, logger, stripUrlQueryAndFragment } from '@sentry/utils'; // Types vendored from @remix-run/server-runtime@1.6.0: // https://github.com/remix-run/remix/blob/f3691d51027b93caa3fd2cdfe146d7b62a6eb8f2/packages/remix-server-runtime/server.ts @@ -134,7 +134,7 @@ function wrapRequestHandler(origRequestHandler: RequestHandler): RequestHandler return async function (this: unknown, request: Request, loadContext?: unknown): Promise { const currentScope = getCurrentHub().getScope(); const transaction = startTransaction({ - name: request.url, + name: stripUrlQueryAndFragment(request.url), op: 'http.server', tags: { method: request.method,