darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-09-09 14:49:49 -04:00
Code Issues Packages Projects Releases Wiki Activity

Add better logging message for tech failure

Browse Source 
So we can easily identify which notification ID failed
This commit is contained in:
David McDonald
2020-12-30 17:28:21 +00:00
parent 2480f91667
commit 977554781f
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/celery/provider_tasks.py
View File

@@ -48,7 +48,9 @@ def deliver_email(self, notification_id):
raise NoResultFound() raise NoResultFound()
send_to_providers.send_email_to_provider(notification) send_to_providers.send_email_to_provider(notification)
except EmailClientNonRetryableException as e: except EmailClientNonRetryableException as e:
current_app.logger.exception(e) current_app.logger.exception(
f"Email notification {notification_id} failed: {e}"
)
update_notification_status_by_id(notification_id, 'technical-failure') update_notification_status_by_id(notification_id, 'technical-failure')
except Exception as e: except Exception as e:
try: try: