Skip to content

Commit 9fc7308

Browse files
committed
fix(modules/alb): always non-null value for ssl_policy
1 parent 7d3743a commit 9fc7308

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

modules/alb/main.tf

+5-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,9 @@
11
locals {
2+
# Load Balancer
23
load_balancer_type = "application"
4+
5+
# Load Balancer Listener
6+
default_ssl_policy = "ELBSecurityPolicy-TLS13-1-2-2021-06"
37
}
48

59
################################################################################
@@ -84,7 +88,7 @@ resource "aws_lb_listener" "this" {
8488
certificate_arn = try(each.value.certificate_arn, null)
8589
port = try(each.value.port, null)
8690
protocol = try(each.value.protocol, null)
87-
ssl_policy = try(each.value.ssl_policy, null)
91+
ssl_policy = try(each.value.ssl_policy, each.value.certificate_arn != null ? local.default_ssl_policy : null)
8892

8993
dynamic "default_action" {
9094
for_each = each.value.default_action

modules/alb/variables.tf

+1-1
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ variable "listeners" {
106106
certificate_arn = optional(string)
107107
port = optional(number)
108108
protocol = optional(string, "HTTP")
109-
ssl_policy = optional(string, "ELBSecurityPolicy-TLS13-1-2-2021-06")
109+
ssl_policy = optional(string, null)
110110
tags = optional(map(string), {})
111111
}))
112112
nullable = false

0 commit comments

Comments
 (0)