Merge pull request #8 from rhythmictech/fix/#7

use try instead of ternary, fix #7

Author: sblack4

main.tf

@@ -10,8 +10,8 @@ locals {
   iam_account_principals = formatlist("arn:aws:iam::%s:root", local.account_ids)
 
   # Resolve resource names
-  bucket     = var.remote_bucket == "" ? aws_s3_bucket.this[0].id : var.remote_bucket
-  kms_key_id = var.kms_key_id == "" ? aws_kms_key.this[0].arn : var.kms_key_id
+  bucket     = try(aws_s3_bucket.this[0].id, var.remote_bucket)
+  kms_key_id = try(aws_kms_key.this[0].arn, var.kms_key_id)
 }
 
 resource "aws_s3_bucket" "this" {

outputs.tf

@@ -1,9 +1,9 @@
 output "s3_bucket_backend" {
   description = "S3 bucket"
-  value       = var.remote_bucket == "" ? aws_s3_bucket.this[0].bucket : var.remote_bucket
+  value       = try(aws_s3_bucket.this[0].bucket, var.remote_bucket)
 }
 
 output "kms_key_arn" {
   description = "ARN of KMS Key for S3 bucket"
-  value       = var.kms_key_id == "" ? aws_kms_key.this[0].arn : var.kms_key_id
+  value       = try(aws_kms_key.this[0].arn, var.kms_key_id)
 }