From 0b1ed0c328f2607aff883961e848d4312e5e2fd3 Mon Sep 17 00:00:00 2001 From: "John C. Bland II" Date: Mon, 7 Aug 2023 13:29:38 -0500 Subject: [PATCH] Change `policy` to `source_policy_documents` (#39) * chore: change policy to source_policy_documents * update readme --------- Co-authored-by: Matt Calhoun --- README.md | 4 ---- main.tf | 2 +- 2 files changed, 1 insertion(+), 5 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 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