hashicorp / terraform-aws-vault

A Terraform Module for how to run Vault on AWS using Terraform and Packer
Apache License 2.0
656 stars 465 forks source link

Addresses an issue with logging configuration with improper for_each iterator #242

Open dchernivetsky opened 3 years ago

dchernivetsky commented 3 years ago

238

brikis98 commented 3 years ago

Pre-commit hook failed:

[INFO] Initializing environment for https://github.com/gruntwork-io/pre-commit.
Terraform fmt............................................................Failed
- hook id: terraform-fmt
- exit code: 3

modules/vault-elb/main.tf
--- old/modules/vault-elb/main.tf
+++ new/modules/vault-elb/main.tf
@@ -26,7 +26,7 @@

   # optional access_logs creation  
   dynamic "access_logs" {
-    for_each = var.access_logs == null ? [] : [ var.access_logs ]
+    for_each = var.access_logs == null ? [] : [var.access_logs]

     content {
       # enabled flag is optional. The user can choose to specify or not. If not specified we check if bucket param is specified and assume that

Please run terraform fmt!

hashicorp-cla commented 2 years ago

CLA assistant check
All committers have signed the CLA.