Skip to content

Commit

Permalink
Revert cloudwatch bucket policy (pushing those changes in separate PR).
Browse files Browse the repository at this point in the history
  • Loading branch information
Matthew-Grayson committed Oct 2, 2023
1 parent 7d626ff commit e88977f
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions infrastructure/cloudwatch.tf
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ resource "aws_s3_bucket_policy" "cloudwatch_bucket" {
"Service" : "logs.amazonaws.com"
},
"Action" : "s3:GetBucketAcl",
"Resource" : aws_s3_bucket.cloudwatch_bucket.arn
"Resource" : "arn:aws:s3:::${var.cloudwatch_bucket_name}"
},
{
"Sid" : "Allow Cloudwatch to write to bucket",
Expand All @@ -38,10 +38,7 @@ resource "aws_s3_bucket_policy" "cloudwatch_bucket" {
"Service" : "logs.amazonaws.com"
},
"Action" : "s3:PutObject",
"Resource" : [
aws_s3_bucket.cloudwatch_bucket.arn,
"${aws_s3_bucket.cloudwatch_bucket.arn}/*"
],
"Resource" : "arn:aws:s3:::${var.cloudwatch_bucket_name}/*",
"Condition" : {
"StringEquals" : {
"s3:x-amz-acl" : "bucket-owner-full-control"
Expand Down

0 comments on commit e88977f

Please sign in to comment.