fix(remix): Pass transaction name as route to RequestData. (#6276)

getsentry · Nov 24, 2022 · 9f522c9 · 9f522c9
1 parent e97ae86
commit 9f522c9
Showing 1 changed file with 23 additions and 5 deletions.
diff --git a/packages/remix/src/utils/instrumentServer.ts b/packages/remix/src/utils/instrumentServer.ts
@@ -96,7 +96,20 @@ async function captureRemixServerException(err: Error, name: string, request: Re
   const normalizedRequest = normalizeRemixRequest(request as unknown as any);
 
   captureException(isResponse(err) ? await extractResponseError(err) : err, scope => {
-    scope.setSDKProcessingMetadata({ request: normalizedRequest });
+    const activeTransactionName = getActiveTransaction()?.name;
+
+    scope.setSDKProcessingMetadata({
+      request: {
+        ...normalizedRequest,
+        // When `route` is not defined, `RequestData` integration uses the full URL
+        route: activeTransactionName
+          ? {
+              path: activeTransactionName,
+            }
+          : undefined,
+      },
+    });
+
     scope.addEventProcessor(event => {
       addExceptionMechanism(event, {
         type: 'instrument',
@@ -378,19 +391,24 @@ function wrapRequestHandler(origRequestHandler: RequestHandler, build: ServerBui
 
       const normalizedRequest = normalizeRemixRequest(request);
 
+      const url = new URL(request.url);
+      const [name, source] = getTransactionName(routes, url, pkg);
+
       if (scope) {
         scope.setSDKProcessingMetadata({
-          request: normalizedRequest,
+          request: {
+            ...normalizedRequest,
+            route: {
+              path: name,
+            },
+          },
         });
       }
 
       if (!options || !hasTracingEnabled(options)) {
         return origRequestHandler.call(this, request, loadContext);
       }
 
-      const url = new URL(request.url);
-      const [name, source] = getTransactionName(routes, url, pkg);
-
       const transaction = startRequestHandlerTransaction(hub, name, source, {
         headers: {
           'sentry-trace': request.headers.get('sentry-trace') || '',