refactor(modules/alb): remove unnecessary usage of try for existing variables, set default values for name to null and preserve_host_header to true

Author: saumitra-dev

modules/alb/main.tf

@@ -7,9 +7,9 @@ locals {
 ################################################################################
 
 resource "aws_lb" "this" {
-  name                       = try(var.name, null)
-  load_balancer_type         = try(local.load_balancer_type, null)
-  internal                   = try(var.internal, null)
+  name                       = var.name
+  load_balancer_type         = local.load_balancer_type
+  internal                   = var.internal
   subnets                    = var.subnets_ids
   security_groups            = var.security_groups_ids
   preserve_host_header       = var.preserve_host_header

modules/alb/variables.tf

@@ -5,7 +5,7 @@
 variable "name" {
   description = "(Optional) Name of the LB."
   type        = string
-  default     = ""
+  default     = null
 }
 
 variable "internal" {
@@ -28,7 +28,7 @@ variable "subnets_ids" {
 variable "preserve_host_header" {
   description = "(Optional) Whether the Application Load Balancer should preserve the Host header in the HTTP request and send it to the target without any change."
   type        = bool
-  default     = false
+  default     = true
 }
 
 variable "enable_deletion_protection" {