Skip to content

process_ses_results: accept receipt_iso_timestamp, extract delivery/bounce time#4797

Open
risicle wants to merge 1 commit intomainfrom
ris-ses-delivery-times
Open

process_ses_results: accept receipt_iso_timestamp, extract delivery/bounce time#4797
risicle wants to merge 1 commit intomainfrom
ris-ses-delivery-times

Conversation

@risicle
Copy link
Copy Markdown
Member

@risicle risicle commented Mar 30, 2026

https://trello.com/c/Xw98lFJ8/1502-collect-log-delivery-and-receipt-timestamps

Apparent delivery/bounce time is already present in the "response" dict serialized straight from the SES SNS stream, so log this in the same way we do for SMS receipts, with the possibility of doing more with it in future.

Similarly, accept a receipt_iso_timestamp task argument which should (eventually) be used to represent the time our lambda received the receipt and put it in the SQS queue.

@risicle risicle marked this pull request as draft March 30, 2026 10:18
…ounce time

apparent delivery/bounce time is already present in the "response"
dict serialized straight from the SES SNS stream, so log this in
the same way we do for SMS receipts, with the possibility of doing
more with it in future.

similarly, accept a receipt_iso_timestamp task argument which
should be used to represent the time our lambda received the
receipt and put it in the SQS queue.
@risicle risicle force-pushed the ris-ses-delivery-times branch from 0578090 to 0e4890c Compare April 1, 2026 15:13
@risicle risicle marked this pull request as ready for review April 1, 2026 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant