Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Handle notificationType field instead of eventType #5

Merged
merged 2 commits into from
Oct 12, 2023

Conversation

TheDoctor0
Copy link
Contributor

@TheDoctor0 TheDoctor0 commented Sep 26, 2023

AWS recently introduced a different way for handling Bounces and Complaints and changed the expected structure of the data based on whether it is an event or notification.

With the change from this PR, the change will be gracefully handled by populating the possibly missing object property.

@akhan619 akhan619 merged commit c7aee75 into akhan619:main Oct 12, 2023
@akhan619
Copy link
Owner

@TheDoctor0 Sorry I was away for a while and couldn't get to this sooner. Thanks for the PR. Appreciate it!!

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.

2 participants