Skip to content

Commit 0eff677

Browse files
committed
eli-306 adding checkov skips
1 parent 19f90bf commit 0eff677

1 file changed

Lines changed: 10 additions & 0 deletions

File tree

infrastructure/stacks/api-layer/iam_policies.tf

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -128,6 +128,9 @@ resource "aws_iam_role_policy" "external_s3_write_policy" {
128128

129129
## KMS
130130
data "aws_iam_policy_document" "dynamodb_kms_key_policy" {
131+
#checkov:skip=CKV_AWS_111: Root user needs full KMS key management
132+
#checkov:skip=CKV_AWS_356: Root user needs full KMS key management
133+
#checkov:skip=CKV_AWS_109: Root user needs full KMS key management
131134
statement {
132135
sid = "EnableIamUserPermissions"
133136
effect = "Allow"
@@ -157,6 +160,9 @@ resource "aws_kms_key_policy" "dynamodb_kms_key" {
157160
}
158161

159162
data "aws_iam_policy_document" "s3_rules_kms_key_policy" {
163+
#checkov:skip=CKV_AWS_111: Root user needs full KMS key management
164+
#checkov:skip=CKV_AWS_356: Root user needs full KMS key management
165+
#checkov:skip=CKV_AWS_109: Root user needs full KMS key management
160166
statement {
161167
sid = "EnableIamUserPermissions"
162168
effect = "Allow"
@@ -186,6 +192,10 @@ resource "aws_kms_key_policy" "s3_rules_kms_key" {
186192
}
187193

188194
data "aws_iam_policy_document" "s3_audit_kms_key_policy" {
195+
#checkov:skip=CKV_AWS_111: Root user needs full KMS key management
196+
#checkov:skip=CKV_AWS_356: Root user needs full KMS key management
197+
#checkov:skip=CKV_AWS_109: Root user needs full KMS key management
198+
189199
statement {
190200
sid = "EnableIamUserPermissions"
191201
effect = "Allow"

0 commit comments

Comments
 (0)