Skip to content

Commit

Permalink
🔇(sentry) active recursive scrubbing of events
Browse files Browse the repository at this point in the history
Some PII are still sent to Sentry, under extra.celery-job.args or kwargs.
The Sentry SDK event scrubber does not scrub recursively by default.
Activating recursive scrubbing to avoid sending emails and usernames to Sentry.
  • Loading branch information
wilbrdt committed Dec 6, 2024
1 parent e98fffb commit 70796dd
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ and this project adheres to
### Changed

- Disable Brevo request on empty Brevo API URL setting
- Activate recursive scrubbing of Sentry events

### Fixed

Expand Down
2 changes: 1 addition & 1 deletion src/app/mork/celery/celery_app.py
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ def init_sentry(**_kwargs):
environment=settings.SENTRY_EXECUTION_ENVIRONMENT,
max_breadcrumbs=50,
send_default_pii=False,
event_scrubber=EventScrubber(pii_denylist=pii_denylist),
event_scrubber=EventScrubber(pii_denylist=pii_denylist, recursive=True),
before_send=before_send,
)
sentry_sdk.set_tag("application", "celery")
Expand Down

0 comments on commit 70796dd

Please sign in to comment.