fix(node): attach local req scope to req.sentryScope#4627
fix(node): attach local req scope to req.sentryScope#4627derN3rd wants to merge 1 commit intogetsentry:mastergetsentry/sentry-javascript:masterfrom
Conversation
|
I guess this should be also added to the docs, if this will make it into master. Will add a PR to sentry-docs afterwards |
|
Hey thanks for opening a PR! We're gonna chat about this as a team (pinged everyone as appropriate), and then we can come back and figure out how to proceed |
|
We are currently discussing if this would be beneficial over Thoughts? |
|
Didn't notice that Sentry.getCurrentHub() is domain-aware in node envs, will try that. If I remember correctly, I saw some issues with that in async contexts, like express-apis are, also here in the repo. Thanks! |
This PR:
Fixes #4607