cloudposse / terraform-aws-cloudfront-s3-cdn

Terraform module to easily provision CloudFront CDN backed by an S3 origin
https://cloudposse.com/accelerate
Apache License 2.0
278 stars 247 forks source link

BucketOwnerEnforced s3 buckets can't have an acl #301

Closed mfuhrmeisterDM closed 8 months ago

mfuhrmeisterDM commented 8 months ago

what

disable creating acl resource in that case.

why

BucketOwnerEnforeced s3 bucket can't have an acl.

references

mergify[bot] commented 8 months ago

Thanks @mfuhrmeisterDM for creating this pull request!

A maintainer will review your changes shortly. Please don't be discouraged if it takes a while.

While you wait, make sure to review our contributor guidelines.

[!TIP]

Need help or want to ask for a PR review to be expedited?

Join us on Slack in the #pr-reviews channel.

mergify[bot] commented 8 months ago

[!IMPORTANT]

Cloud Posse Engineering Team Review Required

This pull request modifies files that require Cloud Posse's review. Please be patient, and a core maintainer will review your changes.

To expedite this process, reach out to us on Slack in the #pr-reviews channel.

kevcube commented 8 months ago

/terratest

osterman commented 8 months ago

/terratest

osterman commented 8 months ago

/terratest