Skip to content

Add logger warnings for display name filter action#113168

Merged
geoffg-sentry merged 2 commits intomasterfrom
Add-logger-warning-for-blocked-display-names
Apr 16, 2026
Merged

Add logger warnings for display name filter action#113168
geoffg-sentry merged 2 commits intomasterfrom
Add-logger-warning-for-blocked-display-names

Conversation

@geoffg-sentry
Copy link
Copy Markdown
Contributor

@geoffg-sentry geoffg-sentry commented Apr 16, 2026

After #113106, adding some logger warning for when the filter comes into action. sentry.security and spam.display-name-blocked.

Also now passing request context in organization_index and passes some from sentry_app and sentry_app_details so we can capture so additional metadata during failed attempts.

@geoffg-sentry geoffg-sentry marked this pull request as ready for review April 16, 2026 14:43
@geoffg-sentry geoffg-sentry requested review from a team as code owners April 16, 2026 14:43
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 16, 2026
Copy link
Copy Markdown
Contributor

@cursor cursor Bot left a comment

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 89613d0. Configure here.

Comment thread src/sentry/sentry_apps/api/parsers/sentry_app.py
@geoffg-sentry geoffg-sentry merged commit e66f412 into master Apr 16, 2026
59 checks passed
@geoffg-sentry geoffg-sentry deleted the Add-logger-warning-for-blocked-display-names branch April 16, 2026 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants