Check if S3 bucket exist to use cloudfront logging

Signed-off-by: Arthur Diniz <[email protected]>
DNXLabs · Mar 6, 2020 · 600f7a5 · 600f7a5
1 parent 58b4a6b
commit 600f7a5
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/_variables.tf b/_variables.tf
@@ -16,9 +16,13 @@ variable "hostname_create" {
 }
 
 variable "cloudfront_logging_bucket" {
+  type        = string
+  default     = null
   description = "Bucket to store logs from app"
 }
 
 variable "cloudfront_logging_prefix" {
+  type        = string
+  default     = ""
   description = "Logging prefix"
 }
diff --git a/cloudfront.tf b/cloudfront.tf
@@ -19,10 +19,14 @@ resource "aws_cloudfront_distribution" "default" {
     }
   }
 
-  logging_config {
-    include_cookies = false
-    bucket          = "${var.cloudfront_logging_bucket}"
-    prefix          = "${var.cloudfront_logging_prefix}"
+  dynamic "logging_config" {
+    for_each = "${compact([var.cloudfront_logging_bucket])}"
+
+    content {
+      include_cookies = false
+      bucket          = "${var.cloudfront_logging_bucket}"
+      prefix          = "${var.cloudfront_logging_prefix}"
+    }
   }
 
   default_cache_behavior {