Skip to content

Commit 4b86009

Browse files
authored
Merge pull request #1545 from maykinmedia/swr/bump-sentry-sdk
Bump Sentry to latest version
2 parents 27bc304 + 31045d1 commit 4b86009

File tree

6 files changed

+18
-5
lines changed

6 files changed

+18
-5
lines changed

requirements/base.in

+1-1
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ djangorestframework-camel-case
7373

7474
# WSGI servers & monitoring - production oriented
7575
uwsgi
76-
sentry-sdk # error monitoring
76+
sentry-sdk[django] # error monitoring
7777
elastic-apm # Elastic APM integration
7878

7979
beautifulsoup4

requirements/base.txt

+2-1
Original file line numberDiff line numberDiff line change
@@ -146,6 +146,7 @@ django==4.2.16
146146
# mozilla-django-oidc
147147
# mozilla-django-oidc-db
148148
# notifications-api-common
149+
# sentry-sdk
149150
# zgw-consumers
150151
# zgw-consumers-oas
151152
django-admin-index==3.1.0
@@ -523,7 +524,7 @@ requests==2.31.0
523524
# messagebird
524525
# mozilla-django-oidc
525526
# zgw-consumers
526-
sentry-sdk==1.38.0
527+
sentry-sdk[django]==2.19.2
527528
# via -r requirements/base.in
528529
simplejson==3.19.2
529530
# via mail-parser

requirements/ci.txt

+3-1
Original file line numberDiff line numberDiff line change
@@ -238,6 +238,7 @@ django==4.2.16
238238
# mozilla-django-oidc
239239
# mozilla-django-oidc-db
240240
# notifications-api-common
241+
# sentry-sdk
241242
# zgw-consumers
242243
# zgw-consumers-oas
243244
django-admin-index==3.1.0
@@ -992,10 +993,11 @@ requests==2.31.0
992993
# zgw-consumers
993994
requests-mock==1.12.1
994995
# via -r requirements/test-tools.in
995-
sentry-sdk==1.38.0
996+
sentry-sdk[django]==2.19.2
996997
# via
997998
# -c requirements/base.txt
998999
# -r requirements/base.txt
1000+
# sentry-sdk
9991001
simplejson==3.19.2
10001002
# via
10011003
# -c requirements/base.txt

requirements/dev.txt

+3-1
Original file line numberDiff line numberDiff line change
@@ -278,6 +278,7 @@ django==4.2.16
278278
# mozilla-django-oidc
279279
# mozilla-django-oidc-db
280280
# notifications-api-common
281+
# sentry-sdk
281282
# zgw-consumers
282283
# zgw-consumers-oas
283284
django-admin-index==3.1.0
@@ -1148,10 +1149,11 @@ requests-mock==1.12.1
11481149
# -r requirements/ci.txt
11491150
roundrobin==0.0.4
11501151
# via locust
1151-
sentry-sdk==1.38.0
1152+
sentry-sdk[django]==2.19.2
11521153
# via
11531154
# -c requirements/ci.txt
11541155
# -r requirements/ci.txt
1156+
# sentry-sdk
11551157
simplejson==3.19.2
11561158
# via
11571159
# -c requirements/ci.txt

src/open_inwoner/celery.py

+6
Original file line numberDiff line numberDiff line change
@@ -70,3 +70,9 @@ def worker_shutdown(**_):
7070

7171

7272
app.steps["worker"].add(LivenessProbe)
73+
74+
75+
@app.task
76+
def trigger_exception():
77+
"""Trigger an exception for debugging purposes."""
78+
return 1 / 0

src/open_inwoner/conf/utils.py

+3-1
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,9 @@ def get_sentry_integrations() -> list:
4646
try:
4747
from sentry_sdk.integrations import celery
4848
except DidNotEnable: # happens if the celery import fails by the integration
49-
pass
49+
logger.warning(
50+
"Unable to initialize Sentry with Celery integration due to failing import"
51+
)
5052
else:
5153
extra.append(celery.CeleryIntegration())
5254

0 commit comments

Comments
 (0)