fix: Allow setup eventbridge without notifications (#160)

31b8e9dc · Pavel · GitHub · 8bd01014 · 31b8e9dc
Commit 31b8e9dc authored May 18, 2022 by Pavel Committed by GitHub May 18, 2022
Show whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

main.tf modules/notification/main.tf +1 -1

No files found.
--- a/modules/notification/main.tf
+++ b/modules/notification/main.tf
@@ -7,7 +7,7 @@ locals {
 }

 resource "aws_s3_bucket_notification" "this" {
-  count = var.create && (length(var.lambda_notifications) > 0 || length(var.sqs_notifications) > 0 || length(var.sns_notifications) > 0) ? 1 : 0
+  count = var.create ? 1 : 0

  bucket = var.bucket