FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

feat(integrations): Add span for DRF authentication by mgaligniana · Pull Request #6984 · getsentry/sentry-python · GitHub

feat(integrations): Add span for DRF authentication - #6984

Merged
alexander-alderman-webb merged 1 commit into
getsentry:masterfrom
mgaligniana:GH-3006-add-span-around-drf-authentication
Aug 7, 2026
Merged

feat(integrations): Add span for DRF authentication#6984
alexander-alderman-webb merged 1 commit into
getsentry:masterfrom
mgaligniana:GH-3006-add-span-around-drf-authentication

Conversation

mgaligniana commented Aug 2, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

Description

In this PR I applied the suggested changes adding a new span in rest_framework.request.Request._authenticate which loops over all configured authentications and making auth queries being in a separate span to reduce visual noise.

  • AI tools were used, I have disclosed which ones, and fully reviewed and verified their output.

Issues

mgaligniana requested a review from a team as a code owner August 2, 2026 02:02
alexander-alderman-webb merged commit baacd47 into getsentry:master Aug 7, 2026
266 of 268 checks passed

Copy link
Copy Markdown
Contributor Author

Thank you!!

mgaligniana added a commit to mgaligniana/sentry-python that referenced this pull request Aug 9, 2026
Add a new span to trace the execution of `rest_framework.request.Request._authenticate()`.
This function loops over all configured authentications. Separating auth queries into a separate span reduces visual noise.
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add a span around Django rest framework authentication

2 participants


Back | FazBrowse Home | New Git URL