Skip to content

Commit 74255a0

Browse files
committed
Limit access to KMS for Wiz
While we are not yet using KMS for any sensitive workloads, we are concerned that we might leak secrets with the current permissions if we ever start to use it. See #336 for a discussion of these concerns.
1 parent 72a9aae commit 74255a0

File tree

1 file changed

+0
-15
lines changed

1 file changed

+0
-15
lines changed

terragrunt/modules/wiz/main.tf

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -121,8 +121,6 @@ resource "aws_iam_role_policy" "tf-policy" {
121121
"Action" : [
122122
"ec2:CopySnapshot",
123123
"ec2:CreateSnapshot",
124-
"kms:CreateKey",
125-
"kms:DescribeKey",
126124
"ec2:GetEbsEncryptionByDefault",
127125
"ec2:DescribeSnapshots"
128126
],
@@ -144,19 +142,6 @@ resource "aws_iam_role_policy" "tf-policy" {
144142
"arn:aws:kms:*:*:key/*"
145143
]
146144
},
147-
{
148-
"Action" : [
149-
"kms:CreateGrant",
150-
"kms:ReEncryptFrom"
151-
],
152-
"Condition" : {
153-
"StringLike" : {
154-
"kms:ViaService" : "ec2.*.amazonaws.com"
155-
}
156-
},
157-
"Effect" : "Allow",
158-
"Resource" : "*"
159-
},
160145
{
161146
"Action" : [
162147
"kms:GetKeyPolicy",

0 commit comments

Comments
 (0)