Skip to content
This repository has been archived by the owner on Nov 10, 2022. It is now read-only.

JSON Message Missing Vital Fields #17

Closed
thenewguy opened this issue Sep 20, 2021 · 0 comments · Fixed by thenewguy/django-bouncy#4
Closed

JSON Message Missing Vital Fields #17

thenewguy opened this issue Sep 20, 2021 · 0 comments · Fixed by thenewguy/django-bouncy#4

Comments

@thenewguy
Copy link

I have recently observed notifications are sent with the key eventType instead of notificationType. The SES docs still state to expect notificationType per https://docs.aws.amazon.com/ses/latest/DeveloperGuide/notification-contents.html

However, I've found other issues related to the problem:

An interesting comment at mautic/mautic#8903 (comment):

If you are using a SES Configuration Set (by setting the custom mail header X-SES-CONFIGURATION-SET), the bounce event generated by SES is slightly different than if you configure bounce management globally. In fact, the difference is just a JSON key. It is eventType instead of notificationType.

We are using the X-SES-CONFIGURATION-SET header. As such, it seems like we need to support both.

The simplest solution seems to be copying eventType to notificationType if notificationType is not set.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant