-
-
Notifications
You must be signed in to change notification settings - Fork 167
feat!: Add support for SNS topic filtering, subscriptions, and topic policy creation #36
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
feat!: Add support for SNS topic filtering, subscriptions, and topic policy creation #36
Conversation
|
opening as draft because I want to update SQS as well and make sure the two work well together for pub/sub architectures (avoid chicken vs egg or scenario when passing attributes from one to the other, etc.) |
antonbabenko
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! Minor comment.
Co-authored-by: Anton Babenko <anton@antonbabenko.com>
## [5.0.0](v4.1.0...v5.0.0) (2023-01-03) ### ⚠ BREAKING CHANGES * Add support for SNS topic filtering, subscriptions, and topic policy creation (#36) ### Features * Add support for SNS topic filtering, subscriptions, and topic policy creation ([#36](#36)) ([707b407](707b407)) ### Bug Fixes * Use a version for to avoid GitHub API rate limiting on CI workflows ([#39](#39)) ([117400e](117400e))
|
This PR is included in version 5.0.0 🎉 |
|
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Description
create_sns_topichas been renamed tocreatepolicyhas been renamed totopic_policyname_prefixhas been replaced with the combination ofnameanduse_name_prefix = trueto ensure only one value is provided*_failure_feedback_role_arn,*_success_feedback_role_arn,*_success_feedback_sample_ratevariables have been replaced with a respective top level variable that contains a map to the three attributes. See before and after below for further clarification.sns_prefix stripped from their namesMotivation and Context
Breaking Changes
UPGRADE-5.0.mdguide providedHow Has This Been Tested?
examples/*to demonstrate and validate my change(s)examples/*projectspre-commit run -aon my pull request