search

cloudposse / terraform-aws-waf

https://cloudposse.com/accelerate
Apache License 2.0
41 stars 58 forks source link

Changes made to fix bug in the module example. #21

Closed askanonymous closed 1 year ago

askanonymous commented 1 year ago

This fix is made for issue #20 . The example for the module was not working so I went through the terraform code to see the issue. In context file, the name is being passed as null. The null value is not accepted for the : metric name as well as the WAF name as both are mandatory. Kindly review the code.

askanonymous commented 1 year ago

Sorry I am a new to public PR. Apologies for the wrong type of PR. I will read more.