Skip to content
This repository has been archived by the owner on Dec 1, 2023. It is now read-only.

Commit

Permalink
Update s3.tf
Browse files Browse the repository at this point in the history
  • Loading branch information
vincentkoc committed Nov 24, 2023
1 parent 6e04ff6 commit 3c0f31e
Showing 1 changed file with 19 additions and 19 deletions.
38 changes: 19 additions & 19 deletions infrastructure/terraform/storage/s3.tf
Original file line number Diff line number Diff line change
Expand Up @@ -18,23 +18,23 @@ resource "aws_s3_bucket_website_configuration" "s3_website_config" {
}
}

resource "aws_s3_bucket_policy" "s3_bucket_policy" {
bucket = aws_s3_bucket.s3_bucket_static_website.id
# resource "aws_s3_bucket_policy" "s3_bucket_policy" {
# bucket = aws_s3_bucket.s3_bucket_static_website.id

#TODO: Update role/principal, used dummy values to get TF working
policy = <<POLICY
{
"Version": "2012-10-17",
"Statement": [
{
"Effect": "Allow",
"Principal": {
"AWS": "arn:aws:iam::123456789012:root"
},
"Action": "s3:GetObject",
"Resource": "arn:aws:s3:::wombat-warden-s3-static-website/*"
}
]
}
POLICY
}
# #TODO: Update role/principal, used dummy values to get TF working
# policy = <<POLICY
# {
# "Version": "2012-10-17",
# "Statement": [
# {
# "Effect": "Allow",
# "Principal": {
# "AWS": "arn:aws:iam::123456789012:root"
# },
# "Action": "s3:GetObject",
# "Resource": "arn:aws:s3:::wombat-warden-s3-static-website/*"
# }
# ]
# }
# POLICY
# }

0 comments on commit 3c0f31e

Please sign in to comment.