search

scribd / terraform-aws-datadog

Terraform module for setting up AWS Datadog integration
https://registry.terraform.io/modules/scribd/datadog/aws/1.0.0
MIT License
38 stars 30 forks source link

Fix `statement_id` length #54

Closed jjkoh95 closed 1 year ago

jjkoh95 commented 1 year ago

While typical statement_id is generated by terraform -- https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/lambda_permission.html

if log-groups are too lengthy, this will cause issue where statement_id is more than 100 characters, refer - https://docs.aws.amazon.com/lambda/latest/dg/API_AddPermission.html#API_AddPermission_RequestSyntax

jim80net commented 1 year ago

Thanks for the contribution @jjkoh95!

jim80net commented 1 year ago

I rebased the last commit since the commit wasn't formatted correctly for the release process.