From 101235f4be8e49894a57af21218124972855ffcb Mon Sep 17 00:00:00 2001 From: "John C. Bland II" Date: Thu, 3 Aug 2023 15:07:08 -0500 Subject: [PATCH 1/2] chore: change policy to source_policy_documents --- main.tf | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/main.tf b/main.tf index de8959a..08dc0bf 100644 --- a/main.tf +++ b/main.tf @@ -32,7 +32,7 @@ module "storage" { restrict_public_buckets = true access_log_bucket_name = var.access_log_bucket_name allow_ssl_requests_only = var.allow_ssl_requests_only - policy = join("", data.aws_iam_policy_document.aws_config_bucket_policy[*].json) + source_policy_documents = [join("", data.aws_iam_policy_document.aws_config_bucket_policy[*].json)] bucket_notifications_enabled = var.bucket_notifications_enabled bucket_notifications_type = var.bucket_notifications_type From 963498bfd04d870f47f99b7d7d9b77e5d5af8015 Mon Sep 17 00:00:00 2001 From: Matt Calhoun Date: Mon, 7 Aug 2023 14:01:36 -0400 Subject: [PATCH 2/2] update readme --- README.md | 4 ---- 1 file changed, 4 deletions(-) diff --git a/README.md b/README.md index 815c09c..d613924 100644 --- a/README.md +++ b/README.md @@ -97,10 +97,6 @@ We highly recommend that in your code you pin the version to the exact version y using so that your infrastructure remains stable, and update versions in a systematic way so that they do not catch you by surprise. -Also, because of a bug in the Terraform registry ([hashicorp/terraform#21417](https://github.com/hashicorp/terraform/issues/21417)), -the registry shows many of our inputs as required when in fact they are optional. -The table below correctly indicates which inputs are required. - Here's how to invoke this example module in your projects