fix(grpc): add isolation_scope to async server interceptor by robinvd · Pull Request #5940 · getsentry/sentry-python
Navigation Menu
{{ message }}
getsentry / sentry-python Public
- Notifications You must be signed in to change notification settings
- Fork 602
Open
fix(grpc): add isolation_scope to async server interceptor#5940
robinvd wants to merge 1 commit intogetsentry:masterfrom
fix(grpc): add isolation_scope to async server interceptor#5940
robinvd wants to merge 1 commit intogetsentry:masterfrom
Conversation
Copy link Copy Markdown
robinvd
commented
Apr 2, 2026
robinvd
commented
Description
Fixes #5894
The async gRPC server interceptor was missing isolation_scope() wrapping, causing scope state to leak between concurrent requests. The sync interceptor already had this. This PR adds the missing isolation scope to the async unary_unary handler and adds a test proving request isolation.
Issues
Reminders
- Please add tests to validate your changes, and lint your code using
tox -e linters. - Add GH Issue ID & Linear ID (if applicable)
- PR title should use conventional commit style (
feat:,fix:,ref:,meta:) - For external contributors: CONTRIBUTING.md, Sentry SDK development docs, Discord community
Copy link Copy Markdown
Contributor
github-actions
bot
commented
Apr 2, 2026
github-actions bot commented
Apr 2, 2026Semver Impact of This PR
🟢 Patch (bug fixes)
📋 Changelog Preview
This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).
New Features ✨
- (integrations) Instrument pyreqwest tracing by servusdei2018 in
#5682
Bug Fixes 🐛
- (grpc) Add isolation_scope to async server interceptor by robinvd in
#5940
Internal Changes 🔧
- (opentelemetry) Ignore mypy error by alexander-alderman-webb in
#5927 - Fix license metadata in setup.py by sl0thentr0py in
#5934 - Update validate-pr workflow by stephanie-anderson in
#5931
🤖 This preview updates automatically when you update the PR.
robinvd
marked this pull request as ready for review
robinvd
requested a review
from a team
as a code owner
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
1 participant