You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hey, thanks for reporting this.
It seems there are still some rough spots in the new express instrumentation. We'll work on updating this to get back to full feature parity there!
cc @onurtemizkan what is the status for this, do you remember?
Yeah we validated this with https://github.com/getsentry/chartcuterie/ - an express service we run in Sentry itself. We'll need to make some upstream changes to OpenTelemetry to fix this!
AbhiPrasad
changed the title
URL parameterization not taking into account routers in v8
URL parameterization not taking into express account routers in v8
May 29, 2024
Is there an existing issue for this?
How do you use Sentry?
Sentry Saas (sentry.io)
Which SDK are you using?
@sentry/node
SDK Version
8.0.0
Framework Version
Node: 20.12.2
Express: 4.19.2
Link to Sentry event
https://fl101.sentry.io/performance/trace/9bd6ae8a599b0919ccd1941defbcb6f5
SDK Setup
Steps to Reproduce
Mount a router with the following:
Expected Result
Calls to
GET /api/foo
should be parameterized as such.Trace collected with 7.x: https://fl101.sentry.io/performance/trace/9f106d65ff644716b32603965c85b61c/?node=span-a36c5a350b874100&node=txn-98cdcb0f6770474aaa27709f1adfe49b&statsPeriod=14d×tamp=1715939877
Actual Result
They are presented as
/foo
.Trace collected with 8.x: https://fl101.sentry.io/performance/trace/9bd6ae8a599b0919ccd1941defbcb6f5/?fov=0%2C112.99991607666016&node=txn-3ead7993a2bf44b99dc67e857ae743ec&statsPeriod=14d×tamp=1715938309
The text was updated successfully, but these errors were encountered: