Merge pull request #138 from schubergphilis/fvb/logging-tweaks

fatbasstard · web-flow · commit 908b274c65b1 · 2025-10-27T15:33:01.000+01:00
fix: Fix logging path
diff --git a/logging.tf b/logging.tf
@@ -17,7 +17,7 @@ resource "aws_cloudwatch_log_delivery_destination" "access_logs" {
   tags          = var.tags
 
   delivery_destination_configuration {
-    destination_resource_arn = "arn:aws:s3:::${var.logging.target_bucket}"
+    destination_resource_arn = "arn:aws:s3:::${var.logging.target_bucket}/${var.logging.target_prefix}"
   }
 }
 
@@ -28,8 +28,4 @@ resource "aws_cloudwatch_log_delivery" "access_logs" {
   delivery_source_name     = aws_cloudwatch_log_delivery_source.access_logs[0].name
   delivery_destination_arn = aws_cloudwatch_log_delivery_destination.access_logs[0].arn
   tags                     = var.tags
-
-  s3_delivery_configuration {
-    suffix_path = var.logging.target_prefix
-  }
 }

Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@ resource "aws_cloudwatch_log_delivery_destination" "access_logs" {`
`17`	`17`	`tags = var.tags`
`18`	`18`
`19`	`19`	`delivery_destination_configuration {`
`20`		`- destination_resource_arn = "arn:aws:s3:::${var.logging.target_bucket}"`
	`20`	`+ destination_resource_arn = "arn:aws:s3:::${var.logging.target_bucket}/${var.logging.target_prefix}"`
`21`	`21`	`}`
`22`	`22`	`}`
`23`	`23`
`@@ -28,8 +28,4 @@ resource "aws_cloudwatch_log_delivery" "access_logs" {`
`28`	`28`	`delivery_source_name = aws_cloudwatch_log_delivery_source.access_logs[0].name`
`29`	`29`	`delivery_destination_arn = aws_cloudwatch_log_delivery_destination.access_logs[0].arn`
`30`	`30`	`tags = var.tags`
`31`		`-`
`32`		`- s3_delivery_configuration {`
`33`		`- suffix_path = var.logging.target_prefix`
`34`		`- }`
`35`	`31`	`}`