ref(http): migrate fiber and fasthttp to context scope API - #1402
ref(http): migrate fiber and fasthttp to context scope API#1402giortzisg wants to merge 1 commit into
Conversation
c51544f to
a2b75f9
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit a2b75f9. Configure here.
|
|
||
| transaction := sentry.StartTransaction( | ||
| sentry.SetHubOnContext(requestCtx, hub), | ||
| requestCtx, |
There was a problem hiding this comment.
Fiber drops request context isolation
Medium Severity
The Fiber integrations no longer cancel or restore the original user context after the request. The previous middleware derived a cancelable context and put the original value back when the handler returned. Without that, request work waiting on Done is never canceled, and a pooled Fiber context that still carries the prior sentry context can make StartTransaction reuse the last request's transaction and clone its isolation scope.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit a2b75f9. Configure here.


Description
This migrates fasthttp frameworks to the new scopes API
Issues
Changelog Entry Instructions
To add a custom changelog entry, uncomment the section above. Supports:
For more details: custom changelog entries
Reminders
feat:,fix:,ref:,meta:)