Merge pull request #73 from byu-oit/s3-acl

Don't create ACL. It doesn't work anymore.
byu-oit · Apr 25, 2023 · 94d6ad1 · 94d6ad1
2 parents 10c5b2d + ac29b0f
commit 94d6ad1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 8 deletions.
diff --git a/README.md b/README.md
@@ -14,7 +14,7 @@ This lambda function will tell CodeDeploy if the tests pass or fail.
 
 ```hcl
 module "postman_test_lambda" {
-  source = "github.com/byu-oit/terraform-aws-postman-test-lambda?ref=v5.0.2"
+  source = "github.com/byu-oit/terraform-aws-postman-test-lambda?ref=v5.0.3"
   app_name = "simple-example"
   postman_collections = [
     {
@@ -85,7 +85,7 @@ selecting your collection/environment and clicking on the info icon.
 
 ```hcl
 module "postman_test_lambda" {
-  source = "github.com/byu-oit/terraform-aws-postman-test-lambda?ref=v5.0.2"
+  source = "github.com/byu-oit/terraform-aws-postman-test-lambda?ref=v5.0.3"
   app_name = "from-postman-api-example"
   postman_collections = [
     {

diff --git a/main.tf b/main.tf
@@ -50,12 +50,6 @@ resource "aws_s3_bucket" "postman_bucket_logs" {
   tags          = var.tags
 }
 
-resource "aws_s3_bucket_acl" "postman_bucket_logs" {
-  count  = local.using_local_files ? 1 : 0
-  bucket = aws_s3_bucket.postman_bucket_logs[count.index].id
-  acl    = "log-delivery-write"
-}
-
 resource "aws_s3_bucket_lifecycle_configuration" "postman_bucket_logs" {
   count  = local.using_local_files ? 1 : 0
   bucket = aws_s3_bucket.postman_bucket_logs[count.index].id