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

adds missing permissions for encrypted sns topics #185

Conversation

bmbferreira
Copy link
Contributor

@bmbferreira bmbferreira commented Jul 9, 2023

Changes proposed in this pull request:

This pull request adds missing permissions on the role associated with the AWS config to be able to use encrypted SNS topics. When an SNS topic is encrypted, additional permissions need to be added to the role or service that is publishing the messages to SNS.

The documentation on AWS config is not clear about this but you can find other examples on AWS for other services that explicitly say the required permissions to be able to publish messages on encrypted SNS topics. For example:

This pull request also adds an example with an encrypted SNS topic.

@rpdelaney rpdelaney self-assigned this Jul 10, 2023
@bmbferreira
Copy link
Contributor Author

Hi @rpdelaney ! Thanks for fixing the docs 🙏 is anything else missing so we can have this merged? Please let me know. Thank you!!

@rpdelaney rpdelaney requested a review from a team July 12, 2023 14:41
@bmbferreira
Copy link
Contributor Author

hey @rpdelaney! Anything missing to merge this? let me know please. Thank you 🙇

@rpdelaney
Copy link
Contributor

I'm sorry, but I am moving on from Truss. I did create a card on our internal issue tracker, so hopefully someone will act on this soon.

@rpdelaney rpdelaney removed their assignment Sep 23, 2023
@avanti-joshi avanti-joshi merged commit 0f03267 into trussworks:main Dec 12, 2023
1 check passed
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.

None yet

3 participants