Skip to content

Commit fcc913e

Browse files
committed
fix condition statement
1 parent 78e2682 commit fcc913e

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

aws_sra_examples/solutions/guardduty/guardduty_org/templates/sra-guardduty-org-delivery-s3-bucket.yaml

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -166,8 +166,8 @@ Resources:
166166
Condition:
167167
StringLike:
168168
aws:PrincipalServiceName: guardduty.*.amazonaws.com
169-
# StringEquals:
170-
# s3:x-amz-acl: bucket-owner-full-control
169+
StringEquals:
170+
s3:x-amz-acl: bucket-owner-full-control
171171

172172
- Sid: DenyUnencryptedObjectUploads
173173
Effect: Deny
@@ -184,14 +184,13 @@ Resources:
184184
- Sid: DenyUnencryptedObjectUploadsOptinRegions
185185
Effect: Deny
186186
Action: s3:PutObject
187-
Condition:
188-
StringNotEquals:
189-
s3:x-amz-server-side-encryption: aws:kms
190187
Resource: !Sub arn:aws:s3:::${rGuardDutyDeliveryS3Bucket}/*
191188
Principal: '*'
192189
Condition:
193190
StringLike:
194191
aws:PrincipalServiceName: guardduty.*.amazonaws.com
192+
StringNotEquals:
193+
s3:x-amz-server-side-encryption: aws:kms
195194

196195
- Sid: DenyIncorrectEncryptionHeader
197196
Effect: Deny

0 commit comments

Comments
 (0)