Skip to content

Commit 168963c

Browse files
antonpirkersentrivana
authored andcommitted
Fixed bug when cron_jobs is set to None in arq integration (#4115)
Handle `None` values in arq configuration gracefully. Fixes #3827
1 parent 9cfda02 commit 168963c

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

Diff for: sentry_sdk/integrations/arq.py

+5-4
Original file line numberDiff line numberDiff line change
@@ -199,12 +199,13 @@ def _sentry_create_worker(*args, **kwargs):
199199
if isinstance(settings_cls, dict):
200200
if "functions" in settings_cls:
201201
settings_cls["functions"] = [
202-
_get_arq_function(func) for func in settings_cls["functions"]
202+
_get_arq_function(func)
203+
for func in settings_cls.get("functions", [])
203204
]
204205
if "cron_jobs" in settings_cls:
205206
settings_cls["cron_jobs"] = [
206207
_get_arq_cron_job(cron_job)
207-
for cron_job in settings_cls["cron_jobs"]
208+
for cron_job in settings_cls.get("cron_jobs", [])
208209
]
209210

210211
if hasattr(settings_cls, "functions"):
@@ -218,11 +219,11 @@ def _sentry_create_worker(*args, **kwargs):
218219

219220
if "functions" in kwargs:
220221
kwargs["functions"] = [
221-
_get_arq_function(func) for func in kwargs["functions"]
222+
_get_arq_function(func) for func in kwargs.get("functions", [])
222223
]
223224
if "cron_jobs" in kwargs:
224225
kwargs["cron_jobs"] = [
225-
_get_arq_cron_job(cron_job) for cron_job in kwargs["cron_jobs"]
226+
_get_arq_cron_job(cron_job) for cron_job in kwargs.get("cron_jobs", [])
226227
]
227228

228229
return old_create_worker(*args, **kwargs)

0 commit comments

Comments
 (0)