From 227c0ccc877c1fe53766a3e6b1eea120017fcec3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marek=20Such=C3=A1nek?= Date: Mon, 24 Mar 2025 08:23:03 +0100 Subject: [PATCH] fix(docworker): Fix Sentry reporting for document templates --- packages/dsw-document-worker/CHANGELOG.md | 4 ++++ packages/dsw-document-worker/dsw/document_worker/worker.py | 7 ++++++- 2 files changed, 10 insertions(+), 1 deletion(-) diff --git a/packages/dsw-document-worker/CHANGELOG.md b/packages/dsw-document-worker/CHANGELOG.md index 565102c3..f370c96e 100644 --- a/packages/dsw-document-worker/CHANGELOG.md +++ b/packages/dsw-document-worker/CHANGELOG.md @@ -7,6 +7,10 @@ to [Semantic Versioning](https://semver.org/spec/v2.0.0.html). ## [Unreleased] +### Fixed + +- Fixed filtering Sentry events for document template issues + ## [4.16.0] diff --git a/packages/dsw-document-worker/dsw/document_worker/worker.py b/packages/dsw-document-worker/dsw/document_worker/worker.py index edfc67db..6a35472f 100644 --- a/packages/dsw-document-worker/dsw/document_worker/worker.py +++ b/packages/dsw-document-worker/dsw/document_worker/worker.py @@ -330,8 +330,13 @@ def filter_templates(event: sentry.Event, hint: sentry.Hint) -> sentry.Event | N phase = event.get('tags', {}).get('phase') if phase in ('render', 'prepare') and template is not None: template_config = Context.get().app.cfg.templates.get_config(template) - if template_config is not None and not template_config.send_sentry: + send_sentry = template_config.send_sentry if template_config else False + if not send_sentry: + LOG.debug('Skipping Sentry event (template, %s, %s)', + event.get('event_id'), hint) return None + LOG.debug('Sending Sentry event (template, %s, %s)', + event.get('event_id'), hint) return event SentryReporter.filters.append(filter_templates)