Skip to content

Commit be1a2da

Browse files
authored
Fix a bug with process_ses_receipts_tasks logging (#2520)
1 parent 8dd806c commit be1a2da

File tree

1 file changed

+2
-6
lines changed

1 file changed

+2
-6
lines changed

app/celery/process_ses_receipts_tasks.py

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -170,19 +170,15 @@ def update_annual_limit_and_bounce_rate(
170170
seeded_today = get_annual_limit_notifications_v2(notification.service_id)
171171

172172
if not is_success:
173-
current_app.logger.info(f"{log_prefix} Delivery failed with error: {aws_response_dict["message"]}")
173+
current_app.logger.info(f"{log_prefix} Delivery failed with error: {aws_response_dict['message']}")
174174

175175
if ff_annual_limit and not seeded_today:
176176
annual_limit_client.increment_email_failed(notification.service_id)
177177
current_app.logger.info(
178178
f"Incremented email_failed count in Redis. Service: {notification.service_id} Notification: {notification.id} Current counts: {annual_limit_client.get_all_notification_counts(notification.service_id)}"
179179
)
180180
else:
181-
current_app.logger.info(
182-
f"{log_prefix} Delivery status: {new_status}" "SES callback return status of {} for notification: {}".format(
183-
new_status, notification.id
184-
)
185-
)
181+
current_app.logger.info(f"{log_prefix} Delivery status: {new_status}")
186182

187183
if ff_annual_limit and not seeded_today:
188184
annual_limit_client.increment_email_delivered(notification.service_id)

0 commit comments

Comments
 (0)