Skip to content

ref(http): migrate fiber and fasthttp to context scope API - #1402

Open
giortzisg wants to merge 1 commit into
scopes/http-framework-integrationsfrom
scopes/fiber-fasthttp-integrations
Open

ref(http): migrate fiber and fasthttp to context scope API#1402
giortzisg wants to merge 1 commit into
scopes/http-framework-integrationsfrom
scopes/fiber-fasthttp-integrations

Conversation

@giortzisg

Copy link
Copy Markdown
Contributor

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:

  • Single entry: just write text
  • Multiple entries: use bullet points
  • Nested bullets: indent 4+ spaces

For more details: custom changelog entries

Reminders

@giortzisg
giortzisg force-pushed the scopes/fiber-fasthttp-integrations branch from c51544f to a2b75f9 Compare August 31, 2026 10:50
@giortzisg
giortzisg marked this pull request as ready for review August 31, 2026 10:52

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ 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.

Comment thread fiber/sentryfiber.go

transaction := sentry.StartTransaction(
sentry.SetHubOnContext(requestCtx, hub),
requestCtx,

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit a2b75f9. Configure here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant