Commit f3f8fe50 authored by Karan Thanvi's avatar Karan Thanvi Committed by GitHub

fix: Using required arguments instead of lookup in replication_configuration (#35)

parent 0e53fa7b
...@@ -117,13 +117,13 @@ resource "aws_s3_bucket" "this" { ...@@ -117,13 +117,13 @@ resource "aws_s3_bucket" "this" {
id = lookup(rules.value, "id", null) id = lookup(rules.value, "id", null)
priority = lookup(rules.value, "priority", null) priority = lookup(rules.value, "priority", null)
prefix = lookup(rules.value, "prefix", null) prefix = lookup(rules.value, "prefix", null)
status = lookup(rules.value, "status", null) status = rules.value.status
dynamic "destination" { dynamic "destination" {
for_each = length(keys(lookup(rules.value, "destination", {}))) == 0 ? [] : [lookup(rules.value, "destination", {})] for_each = length(keys(lookup(rules.value, "destination", {}))) == 0 ? [] : [lookup(rules.value, "destination", {})]
content { content {
bucket = lookup(destination.value, "bucket", null) bucket = destination.value.bucket
storage_class = lookup(destination.value, "storage_class", null) storage_class = lookup(destination.value, "storage_class", null)
replica_kms_key_id = lookup(destination.value, "replica_kms_key_id", null) replica_kms_key_id = lookup(destination.value, "replica_kms_key_id", null)
account_id = lookup(destination.value, "account_id", null) account_id = lookup(destination.value, "account_id", null)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment