-
Notifications
You must be signed in to change notification settings - Fork 0
lambda-kms module #19
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
base: master
Are you sure you want to change the base?
Conversation
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.
Sorry it took me a while to get to this. Thanks for the PR - can you take a look at the comments and make some small changes?
modules/lambda-kms/main.tf
Outdated
@@ -0,0 +1,18 @@ | |||
data "template_file" "kms_policy" { |
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.
The template_file
data resource for IAM policies is an older way of doing this. Can you switch this to use the newer iam_policy_document
data resource? There are a few examples in this repo of how to use it such as this one.
] | ||
}, | ||
"Action": [ | ||
"kms:*" |
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.
I think it might be worthwhile to allow root
and the deployment user access to kms:*
, but also add a deny
on decrypt
(and possibly any other sensitive functions). This will allow the user to administer the key for deployment purposes, but not open up encrypted data to a wider audience than necessary.
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.
Decrypt is needed for fleece config
.
aebba4d
to
7505e0a
Compare
7505e0a
to
7b5463d
Compare
@ryandub per our discussion, I added a
|
No description provided.