Skip to content

Commit 0c60ec0

Browse files
committed
fix: Handle Celery broker errors in rejection notification
Catch all exceptions when calling send_rejection_notification.delay(), not just ImportError. This prevents 500 errors when Celery broker is unavailable. Bump version to 0.6.4
1 parent 969b09d commit 0c60ec0

3 files changed

Lines changed: 4 additions & 2 deletions

File tree

django_forms_workflows/__init__.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
Enterprise-grade, database-driven form builder with approval workflows
44
"""
55

6-
__version__ = "0.6.3"
6+
__version__ = "0.6.4"
77
__author__ = "Django Forms Workflows Contributors"
88
__license__ = "LGPL-3.0-only"
99

django_forms_workflows/views.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -338,6 +338,8 @@ def approve_submission(request, task_id):
338338
send_rejection_notification.delay(submission.id)
339339
except ImportError:
340340
logger.warning("Celery tasks not available, skipping notification")
341+
except Exception as e:
342+
logger.warning("Failed to queue rejection notification: %s", e)
341343
else:
342344
# Approval - check workflow logic
343345
from .workflow_engine import handle_approval

pyproject.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
[tool.poetry]
22
name = "django-forms-workflows"
3-
version = "0.6.3"
3+
version = "0.6.4"
44
description = "Enterprise-grade, database-driven form builder with approval workflows and external data integration"
55
license = "LGPL-3.0-only"
66
readme = "README.md"

0 commit comments

Comments
 (0)