Skip to content

feat: remove global event processors - #2245

Merged
Litarnus merged 2 commits into
5.xfrom
remove-global-event-processors
Oct 7, 2026
Merged

Litarnus merged 2 commits into
5.xfrom
remove-global-event-processors

Conversation

@Litarnus

@Litarnus Litarnus commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Removes the static global processors list in favour of event processors on Scopes. The direct replacement for it is using the global scope directly:

SentrySdk::getGlobalScope()->addEventProcessor(...);

The reason to remove this is that we already have event processors on Scopes, meaning that the static list is redundant.

This PR also introduces a small optimisation and correction: We would run event processors unconditionally for all events, including logs and metric events which use less data than error events. That means that for all those events, we would add data only to discard it while serialising. The event processors are now guarded by type and are only executed for errors and transactions

@Litarnus
Litarnus merged commit 66a1a18 into 5.x Oct 7, 2026
48 checks passed
@Litarnus
Litarnus deleted the remove-global-event-processors branch October 7, 2026 09:55
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