feat: Added control to create logs by Lambda@Edge in all regions

terraform-aws-modules / terraform-aws-lambda

Terraform module, which takes care of a lot of AWS Lambda/serverless tasks (build dependencies, packages, updates, deployments) in countless combinations 🇺🇦

https://registry.terraform.io/modules/terraform-aws-modules/lambda/aws

Apache License 2.0

886 stars 656 forks source link

feat: Added control to create logs by Lambda@Edge in all regions #462

Closed rysi3k closed 1 year ago

rysi3k commented 1 year ago

Description

Added simple flag, for not create IAM policy which allows to create logs by Lambda@Edge in regions other than N.Virginia.

Motivation and Context

In some cases that logs are not necessary, but storing it costs extra $.

Breaking Changes

How Has This Been Tested?

[x] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[x] I have tested and validated these changes using one or more of the provided examples/* projects
[ ] I have executed pre-commit run -a on my pull request

rysi3k commented 1 year ago

@antonbabenko changed according to your suggestions

antonbabenko commented 1 year ago

This PR is included in version 4.18.0 :tada:

github-actions[bot] commented 1 year ago

I'm going to lock this pull request 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 related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.