refactor: remove variables for s3 bucket prefixes access logs and connection logs

Author: saumitra-dev

main.tf

@@ -6,15 +6,17 @@ locals {
   ecs_task_definition_requires_compatibilities = ["EC2"]
 
   # ALB
+  s3_bucket_access_logs_prefix = "alb-access-logs"
   alb_access_logs_default_s3_configuration = var.create_s3_bucket_for_alb_logging ? {
     bucket  = module.s3_bucket[0].bucket_id
     enabled = true
-    prefix  = var.s3_bucket_access_logs_prefix
+    prefix  = local.s3_bucket_access_logs_prefix
   } : null
+  s3_bucket_connection_logs_prefix = "alb-connection-logs"
   alb_connection_logs_default_s3_configuration = var.create_s3_bucket_for_alb_logging ? {
     bucket  = module.s3_bucket[0].bucket_id
     enabled = true
-    prefix  = var.s3_bucket_connection_logs_prefix
+    prefix  = local.s3_bucket_connection_logs_prefix
   } : null
 
   alb_target_groups = {

variables.tf

@@ -163,20 +163,6 @@ variable "s3_elb_service_account_arn" {
   default     = null
 }
 
-variable "s3_bucket_access_logs_prefix" {
-  description = "(Optional) - Prefix for storing ALB access logs in the S3 bucket."
-  type        = string
-  nullable    = false
-  default     = "alb-access-logs"
-}
-
-variable "s3_bucket_connection_logs_prefix" {
-  description = "(Optional) - Prefix for storing ALB connection logs in the S3 bucket."
-  type        = string
-  nullable    = false
-  default     = "alb-connection-logs"
-}
-
 variable "s3_bucket_tags" {
   description = "(Optional) Map of tags to assign to the bucket."
   type        = map(string)