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

Record providers we time out notifications for

Browse Source 
This will help us monitor issues with delivery receipts and keep
track of provider performance over time.

I'm not concerned about performance here:

- The number of notifications to time out is usually small.
- This task only runs once a day.
- Calls to StatsD are quick and cheap.
This commit is contained in:
Ben Thorner
2021-12-14 12:38:25 +00:00
parent 11278c47f5
commit c8cf057eba
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
app/celery/nightly_tasks.py
View File

@@ -8,7 +8,7 @@ from notifications_utils.clients.zendesk.zendesk_client import (
from sqlalchemy import func from sqlalchemy import func
from sqlalchemy.exc import SQLAlchemyError from sqlalchemy.exc import SQLAlchemyError
from app import notify_celery, zendesk_client from app import notify_celery, statsd_client, zendesk_client
from app.aws import s3 from app.aws import s3
from app.config import QueueNames from app.config import QueueNames
from app.cronitor import cronitor from app.cronitor import cronitor
@@ -123,6 +123,7 @@ def timeout_notifications():
notifications = dao_timeout_notifications(cutoff_time) notifications = dao_timeout_notifications(cutoff_time)
for notification in notifications: for notification in notifications:
statsd_client.incr(f'timeout-sending.{notification.sent_by}')
check_and_queue_callback_task(notification) check_and_queue_callback_task(notification)
current_app.logger.info( current_app.logger.info(