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

Add better logging to understand callback delivery cases

Browse Source 
In particular, we want to know how often callbacks arrive before the
notification being persisted
This commit is contained in:
David McDonald
2020-06-04 15:43:03 +01:00
parent 8b7a0b88cb
commit 7bc02ac26e
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
app/celery/process_ses_receipts_tasks.py
View File

@@ -43,10 +43,14 @@ def process_ses_results(self, response):
except NoResultFound:
message_time = iso8601.parse_date(ses_message['mail']['timestamp']).replace(tzinfo=None)
if datetime.utcnow() - message_time < timedelta(minutes=5):
current_app.logger.info(
f"notification not found for reference: {reference} (update to {notification_status}). "
f"Callback may have arrived before notification was persisted to the DB. Adding task to retry queue"
)
self.retry(queue=QueueNames.RETRY)
else:
current_app.logger.warning(
"notification not found for reference: {} (update to {})".format(reference, notification_status)
f"notification not found for reference: {reference} (update to {notification_status})"
)
return