fix: fully disable the canned ACL during bucket creation (#52)

Author: baixiac

config/README.md

@@ -63,7 +63,6 @@
 | [aws_route53_record.smtp_mail_from_txt](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/route53_record) | resource |
 | [aws_route53_record.this](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/route53_record) | resource |
 | [aws_s3_bucket.this](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/s3_bucket) | resource |
-| [aws_s3_bucket_acl.this](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/s3_bucket_acl) | resource |
 | [aws_s3_bucket_ownership_controls.this](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/s3_bucket_ownership_controls) | resource |
 | [aws_s3_bucket_server_side_encryption_configuration.this](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/s3_bucket_server_side_encryption_configuration) | resource |
 | [aws_secretsmanager_secret.dockerhub_credentials](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/secretsmanager_secret) | resource |

config/s3.tf

@@ -19,15 +19,6 @@ resource "aws_s3_bucket_ownership_controls" "this" {
   depends_on = [aws_s3_bucket.this]
 }
 
-resource "aws_s3_bucket_acl" "this" {
-  for_each = { for k, v in local.s3_bucket_names : k => v if var.enable_s3 }
-
-  bucket = aws_s3_bucket.this[each.key].id
-  acl    = "private"
-
-  depends_on = [aws_s3_bucket_ownership_controls.this]
-}
-
 resource "aws_s3_bucket_server_side_encryption_configuration" "this" {
   for_each = { for k, v in local.s3_bucket_names : k => v if var.enable_s3 }