Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

S3 Bucket - acl - Argument is deprecated #120

Open
fracampit opened this issue Dec 17, 2022 · 3 comments
Open

S3 Bucket - acl - Argument is deprecated #120

fracampit opened this issue Dec 17, 2022 · 3 comments
Labels
bug 🐛 An issue with the system

Comments

@fracampit
Copy link

Found a bug? Maybe our Slack Community can help.

Slack Community

Describe the Bug

╷
│ Warning: Argument is deprecated
│
│   with module.build.aws_s3_bucket.cache_bucket,
│   on .terraform\modules\build\main.tf line 11, in resource "aws_s3_bucket" "cache_bucket":
│   11:   acl           = "private"
│
│ Use the aws_s3_bucket_acl resource instead
╵

Expected Behavior

Running the module does not throw any warnings

Steps to Reproduce

Steps to reproduce the behavior:

  1. Run the module

Screenshots

If applicable, add screenshots or logs to help explain your problem.

Environment (please complete the following information):

terraform {
  required_providers {
    aws = {
      source  = "hashicorp/aws"
      version = "~> 4.16"
    }
  }

  required_version = ">= 1.2.0"
}
@fracampit fracampit added the bug 🐛 An issue with the system label Dec 17, 2022
@fracampit
Copy link
Author

The default value for acl is private anyways, so can just remove the line entirely

@clemblanco
Copy link

Bump up

@bt-macole
Copy link
Contributor

Should also be BucketOwnerEnforced

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 An issue with the system
Projects
None yet
Development

No branches or pull requests

3 participants