From f1b05da3db39d3df00682d0d04445a0a135a60b6 Mon Sep 17 00:00:00 2001 From: Anton Babenko Date: Mon, 22 Mar 2021 11:07:23 +0100 Subject: [PATCH 1/2] fix: ACL value can be null even when using Terragrunt --- main.tf | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/main.tf b/main.tf index b3f40e7f..c4b94038 100644 --- a/main.tf +++ b/main.tf @@ -3,7 +3,9 @@ resource "aws_s3_bucket" "this" { bucket = var.bucket bucket_prefix = var.bucket_prefix - acl = var.acl + + # hack when `null` value can't be used (eg, from terragrunt, https://github.com/gruntwork-io/terragrunt/pull/1367) + acl = var.acl != "null"? var.acl : null tags = var.tags force_destroy = var.force_destroy acceleration_status = var.acceleration_status From 4d72c8d8dd43dc0e29c48a8246cdd0e4a9554a26 Mon Sep 17 00:00:00 2001 From: Anton Babenko Date: Mon, 22 Mar 2021 11:12:04 +0100 Subject: [PATCH 2/2] Fixed formatting --- main.tf | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/main.tf b/main.tf index c4b94038..98d288af 100644 --- a/main.tf +++ b/main.tf @@ -1,11 +1,12 @@ resource "aws_s3_bucket" "this" { count = var.create_bucket ? 1 : 0 - bucket = var.bucket - bucket_prefix = var.bucket_prefix + bucket = var.bucket + bucket_prefix = var.bucket_prefix # hack when `null` value can't be used (eg, from terragrunt, https://github.com/gruntwork-io/terragrunt/pull/1367) - acl = var.acl != "null"? var.acl : null + acl = var.acl != "null" ? var.acl : null + tags = var.tags force_destroy = var.force_destroy acceleration_status = var.acceleration_status