chore: consolidate locals

cloudposse · Nov 5, 2021 · e8aaec4 · e8aaec4
1 parent 7372db1
commit e8aaec4
Showing 1 changed file with 2 additions and 5 deletions.
diff --git a/main.tf b/main.tf
@@ -49,6 +49,8 @@ locals {
   logging_bucket_enabled      = local.bucket_enabled && var.logging_bucket_enabled
   logging_bucket_name_default = try(var.logging["bucket_name"], "${local.bucket_name}-logs")
   logging_prefix_default      = try(var.logging["prefix"], "logs/")
+  logging_bucket_name         = local.logging_bucket_enabled ? module.log_storage.bucket_id : local.logging_bucket_name_default
+  logging_prefix              = local.logging_bucket_enabled ? module.log_storage.prefix : local.logging_prefix_default
 }
 
 data "aws_iam_policy_document" "prevent_unencrypted_uploads" {
@@ -151,11 +153,6 @@ module "log_storage" {
   context = module.this.context
 }
 
-locals {
-  logging_bucket_name = local.logging_bucket_enabled ? module.log_storage.bucket_id : local.logging_bucket_name_default
-  logging_prefix      = local.logging_bucket_enabled ? module.log_storage.prefix : local.logging_prefix_default
-}
-
 resource "aws_s3_bucket" "default" {
   count = local.bucket_enabled ? 1 : 0