terraform-aws-modules / terraform-aws-s3-bucket

Terraform module to create AWS S3 resources 🇺🇦

Home Page:https://registry.terraform.io/modules/terraform-aws-modules/s3-bucket/aws

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Add support for NLB access logs

wiegandf opened this issue · comments

Describe the solution you'd like.

Similar to the attach_elb_log_delivery_policy flag, we could have an attach_nlb_log_delivery_policy flag that would account for https://docs.aws.amazon.com/elasticloadbalancing/latest/network/load-balancer-access-logs.html#access-logging-bucket-requirements.

Describe alternatives you've considered.

Since the module provides the option to attach bucket policies, you can create the policy on your own and attach it to the bucket.

Hi @wiegandf !

Along the same lines, there can be support for other log deliveries by other services, such as CloudTrail, Config, etc.

PS: Pull-request is welcome.

v2.4.0 has been just released.

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.