Skip to content

Commit

Permalink
Change policy to source_policy_documents (#39)
Browse files Browse the repository at this point in the history
* chore: change policy to source_policy_documents

* update readme

---------

Co-authored-by: Matt Calhoun <[email protected]>
  • Loading branch information
johncblandii and mcalhoun authored Aug 7, 2023
1 parent 5955c46 commit 0b1ed0c
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 5 deletions.
4 changes: 0 additions & 4 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -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

Expand Down
2 changes: 1 addition & 1 deletion main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 0b1ed0c

Please sign in to comment.