Found a second place without null checking

Michael Chmielewski · Michael Chmielewski · commit 134dd94a2c81 · 2020-10-05T11:10:04.000-04:00
diff --git a/outputs.tf b/outputs.tf
@@ -39,7 +39,7 @@ output "s3_bucket_id" {
 }
 
 output "s3_bucket_arn" {
-  value = coalesce((length(aws_s3_bucket.bucket) > 0 ? aws_s3_bucket.bucket[0].arn : ""), var.existing_cloudtrail.s3_bucket_arn)
+  value = coalesce((length(aws_s3_bucket.bucket) > 0 ? aws_s3_bucket.bucket[0].arn : ""), (var.existing_cloudtrail != null ? var.existing_cloudtrail.s3_bucket_arn : ""))
 }
 
 output "iam_role_name" {

Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ output "s3_bucket_id" {`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`output "s3_bucket_arn" {`
`42`		`- value = coalesce((length(aws_s3_bucket.bucket) > 0 ? aws_s3_bucket.bucket[0].arn : ""), var.existing_cloudtrail.s3_bucket_arn)`
	`42`	`+ value = coalesce((length(aws_s3_bucket.bucket) > 0 ? aws_s3_bucket.bucket[0].arn : ""), (var.existing_cloudtrail != null ? var.existing_cloudtrail.s3_bucket_arn : ""))`
`43`	`43`	`}`
`44`	`44`
`45`	`45`	`output "iam_role_name" {`