Merge pull request #6 from Flaconi/OPS-4317-cloudfront-logging

OPS-4317: Enable Cloudfront Logging
Flaconi · Apr 21, 2021 · 88653d1 · 88653d1
2 parents ab7afb0 + 7df0bf5
commit 88653d1
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 1 deletion.
diff --git a/README.md b/README.md
@@ -16,6 +16,7 @@ This module will create cdn endpoint with alias and SSL-certificate
 | r53\_hostname | Hostname for CloudFront alias | `string` | n/a | yes |
 | r53\_zone\_id | Route53 zone ID to be used for hostname and certificate validation | `string` | n/a | yes |
 | s3\_origin\_hostname | Hostname of S3-bucket to be used as origin | `string` | n/a | yes |
+| cdn\_logging | Prefix in s3 bucket for cdn logs | `string` | `""` | no |
 | s3\_logging\_hostname | Hostname of S3-bucket to be used for logging | `string` | `""` | no |
 | tags | Map of custom tags for the provisioned resources | `map(string)` | `{}` | no |
 

diff --git a/main.tf b/main.tf
@@ -26,7 +26,11 @@ module "cloudfront" {
     s3_bucket = "Access identity for CDN (${var.r53_hostname})"
   }
 
-  logging_config = var.s3_logging_hostname == "" ? {} : { bucket = var.s3_logging_hostname }
+  logging_config = var.s3_logging_hostname == "" ? {} : {
+    bucket          = var.s3_logging_hostname
+    include_cookies = false
+    prefix          = var.cdn_logging
+  }
 
   origin = {
     s3_origin = {

diff --git a/variables.tf b/variables.tf
@@ -9,6 +9,12 @@ variable "s3_logging_hostname" {
   default     = ""
 }
 
+variable "cdn_logging" {
+  description = "Prefix in s3 bucket for cdn logs"
+  type        = string
+  default     = ""
+}
+
 variable "r53_hostname" {
   description = "Hostname for CloudFront alias"
   type        = string