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

Automatically Add kms:Decrypt to iamRoleStatements #23

Open
mccredie opened this issue Aug 27, 2019 · 0 comments
Open

Automatically Add kms:Decrypt to iamRoleStatements #23

mccredie opened this issue Aug 27, 2019 · 0 comments
Labels
good first issue Good for newcomers

Comments

@mccredie
Copy link
Contributor

Is your feature request related to a problem? Please describe.
It is in the example. Basically, it is a step that needs to be performed, and the plugin could easily take care of it for you.

Describe the solution you'd like
The plugin will look at the referenced secrets and determine the ARNs for the CMKs used, and add role statements allowing the lambda kms:Decrypt using those CMKs

Describe alternatives you've considered
We could just print a warning, or fail the packaging command, if we detect that the required statement doesn't exist. This might actually be tricky to figure out.

Additional context
See the example in README.md for the required role statements.

@mccredie mccredie added the good first issue Good for newcomers label Sep 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Development

No branches or pull requests

1 participant