Skip to content

feat(outcomes): Add callback_error discard reason - #8422

Merged
msonnb merged 1 commit into
masterfrom
ms/outcomes-callback-error
Sep 1, 2026
Merged

feat(outcomes): Add callback_error discard reason#8422
msonnb merged 1 commit into
masterfrom
ms/outcomes-callback-error

Conversation

@msonnb

@msonnb msonnb commented Sep 1, 2026

Copy link
Copy Markdown
Member

This adds a new callback_error discard reason for when events are dropped because a user-provided callback (like beforeSend, tracesSampler, etc.) throws an error. Without this a user wouldn't be able to distinguish between an event being filtered because their callback worked as intended or because it threw.

ref https://linear.app/getsentry/project/wrap-all-sdk-user-callbacks-in-a-try-catch-6bfdbe9f7268/overview

@msonnb
msonnb requested a review from a team as a code owner September 1, 2026 13:49
@msonnb
msonnb merged commit 234695b into master Sep 1, 2026
66 checks passed
@msonnb
msonnb deleted the ms/outcomes-callback-error branch September 1, 2026 14:28
msonnb added a commit to getsentry/sentry that referenced this pull request Sep 2, 2026
This adds the new `callback_error` client discard reason to usage stats.
Used in SDKs to indicate that an event was discarded because a
user-provided callback threw an error.

ref getsentry/snuba#8422
ref
https://linear.app/getsentry/project/wrap-all-sdk-user-callbacks-in-a-try-catch-6bfdbe9f7268/overview
msonnb added a commit to getsentry/sentry-docs that referenced this pull request Sep 2, 2026
msonnb added a commit to getsentry/sentry-docs that referenced this pull request Sep 2, 2026
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.

2 participants