From 4c98cfe2bedba1a0ef5eb900fe141d23e320098a Mon Sep 17 00:00:00 2001 From: Abhijeet Prasad Date: Wed, 17 Aug 2022 13:21:16 -0400 Subject: [PATCH] fix(remix): Store transaction on express req --- packages/remix/src/utils/serverAdapters/express.ts | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/packages/remix/src/utils/serverAdapters/express.ts b/packages/remix/src/utils/serverAdapters/express.ts index 04344e99990c..71798863430c 100644 --- a/packages/remix/src/utils/serverAdapters/express.ts +++ b/packages/remix/src/utils/serverAdapters/express.ts @@ -51,7 +51,10 @@ function wrapExpressRequestHandler( } const url = new URL(request.url); - startRequestHandlerTransaction(url, request.method, routes, hub, pkg); + const transaction = startRequestHandlerTransaction(url, request.method, routes, hub, pkg); + // save a link to the transaction on the response, so that even if there's an error (landing us outside of + // the domain), we can still finish it (albeit possibly missing some scope data) + (res as AugmentedExpressResponse).__sentryTransaction = transaction; return origRequestHandler.call(this, req, res, next); }; }