0.1.1 (#5)

petewilcock · web-flow · commit 8d64d81185fc · 2021-06-19T23:11:20.000+01:00
* Fixing WAF active bug
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,6 +4,7 @@
 
 Bugfix: Refactor md5 calculation on archive_file in codebuild child module.
 Bugfix: Re-typed AWS account number as string to avoid rounding on account numbers prepended with zeros.
+Bugfix: Fix passed WAF variable values if set to inactive.
 
 ## 0.1.0 - 19th June 2021
 
diff --git a/main.tf b/main.tf
@@ -29,7 +29,7 @@ module "cloudfront" {
   depends_on = [aws_acm_certificate_validation.wordpress_site,
   module.waf]
   cloudfront_class = var.cloudfront_class
-  waf_acl_arn      = module.waf[0].waf_acl_arn
+  waf_acl_arn      = var.waf_enabled ? module.waf[0].waf_acl_arn : null
 }
 
 module "waf" {

Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ module "cloudfront" {`
`29`	`29`	`depends_on = [aws_acm_certificate_validation.wordpress_site,`
`30`	`30`	`module.waf]`
`31`	`31`	`cloudfront_class = var.cloudfront_class`
`32`		`- waf_acl_arn = module.waf[0].waf_acl_arn`
	`32`	`+ waf_acl_arn = var.waf_enabled ? module.waf[0].waf_acl_arn : null`
`33`	`33`	`}`
`34`	`34`
`35`	`35`	`module "waf" {`