Skip to content

Commit 41c6d0d

Browse files
committed
Fix list check
1 parent bdbc737 commit 41c6d0d

File tree

1 file changed

+8
-8
lines changed
  • operations/deployment/terraform/modules/aws/lb

1 file changed

+8
-8
lines changed

operations/deployment/terraform/modules/aws/lb/aws_lb.tf

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@ locals {
44

55
alb_listen_port_list = var.aws_alb_listen_port != "" ? [for n in split(",", var.aws_alb_listen_port) : tonumber(n)] : []
66

7-
alb_listen_port = var.aws_alb_listen_port != "" ? [for n in split(",", var.aws_alb_listen_port) : tonumber(n)] : var.aws_certificates_selected_arn != "" ? [443] : [80]
8-
alb_listen_protocol = var.aws_alb_listen_protocol != "" ? [for n in split(",", var.aws_alb_listen_protocol) : n] : var.aws_certificates_selected_arn != "" ? ["HTTPS"] : ["HTTP"]
7+
alb_listen_port = var.aws_alb_listen_port != "" ? [for n in split(",", var.aws_alb_listen_port) : tonumber(n)] : var.aws_certificate_enabled ? [443] : [80]
8+
alb_listen_protocol = var.aws_alb_listen_protocol != "" ? [for n in split(",", var.aws_alb_listen_protocol) : n] : var.aws_certificate_enabled ? ["HTTPS"] : ["HTTP"]
99
alb_app_port = var.aws_alb_app_port != "" ? [for n in split(",", var.aws_alb_app_port) : tonumber(n)] : local.alb_listen_port
1010
alb_app_protocol = var.aws_alb_app_protocol != "" ? [for n in split(",", var.aws_alb_app_protocol) : n] : [for _ in local.alb_app_port : "HTTP"]
1111

@@ -150,7 +150,7 @@ resource "aws_alb_listener" "lb_listener" {
150150
}
151151

152152
resource "aws_alb_listener" "http_redirect" {
153-
count = var.aws_alb_redirect_enable && var.aws_certificate_enabled && !contains(local.alb_listen_port_list, 80) ? 1 : 0 #!contains(local.alb_listen_port, 80) ? 1 : 0 : 0
153+
count = var.aws_alb_redirect_enable && var.aws_certificate_enabled && !contains(local.alb_listen_port, 80) ? 1 : 0 #!contains(local.alb_listen_port, 80) ? 1 : 0 : 0
154154
load_balancer_arn = aws_lb.vm_alb.arn
155155
port = "80"
156156
protocol = "HTTP"
@@ -171,7 +171,7 @@ resource "aws_alb_listener" "http_redirect" {
171171
}
172172

173173
resource "aws_alb_listener" "http_forward" {
174-
count = var.aws_alb_redirect_enable && !var.aws_alb_www_to_apex_redirect && !var.aws_certificate_enabled && !contains(local.alb_listen_port_list, 80) ? 1 : 0 #!contains(local.alb_listen_port, 80) ? 1 : 0 : 0
174+
count = var.aws_alb_redirect_enable && !var.aws_alb_www_to_apex_redirect && !var.aws_certificate_enabled && !contains(local.alb_listen_port, 80) ? 1 : 0 #!contains(local.alb_listen_port, 80) ? 1 : 0 : 0
175175
load_balancer_arn = aws_lb.vm_alb.arn
176176
port = "80"
177177
protocol = "HTTP"
@@ -187,7 +187,7 @@ resource "aws_alb_listener" "http_forward" {
187187
}
188188

189189
resource "aws_alb_listener" "http_www_redirect" {
190-
count = var.aws_alb_redirect_enable && var.aws_alb_www_to_apex_redirect && !var.aws_certificate_enabled ? !contains(local.alb_listen_port_list, 80) ? 1 : 0 : 0
190+
count = var.aws_alb_redirect_enable && var.aws_alb_www_to_apex_redirect && !var.aws_certificate_enabled ? !contains(local.alb_listen_port, 80) ? 1 : 0 : 0
191191
load_balancer_arn = aws_lb.vm_alb.arn
192192
port = "80"
193193
protocol = "HTTP"
@@ -250,7 +250,7 @@ resource "aws_lb_listener_rule" "redirect_www_to_apex" {
250250
}
251251

252252
resource "aws_security_group_rule" "incoming_alb_http" {
253-
count = !contains(local.alb_listen_port_list, 80) ? length(aws_alb_listener.http_redirect) + length(aws_alb_listener.http_forward) + length(aws_alb_listener.http_www_redirect) : 0
253+
count = !contains(local.alb_listen_port, 80) ? length(aws_alb_listener.http_redirect) + length(aws_alb_listener.http_forward) + length(aws_alb_listener.http_www_redirect) : 0
254254
type = "ingress"
255255
from_port = 80
256256
to_port = 80
@@ -260,7 +260,7 @@ resource "aws_security_group_rule" "incoming_alb_http" {
260260
}
261261

262262
resource "aws_security_group_rule" "incoming_alb_https" {
263-
count = !contains(local.alb_listen_port_list, 443) ? length(aws_alb_listener.https_redirect) : 0
263+
count = !contains(local.alb_listen_port, 443) ? length(aws_alb_listener.https_redirect) : 0
264264
type = "ingress"
265265
from_port = 443
266266
to_port = 443
@@ -271,7 +271,7 @@ resource "aws_security_group_rule" "incoming_alb_https" {
271271
###
272272

273273
resource "aws_alb_listener" "https_redirect" {
274-
count = var.aws_alb_redirect_enable && var.aws_certificate_enabled && !contains(local.alb_listen_port_list, 443) ? 1 : 0 #!contains(local.alb_listen_port, 443) ? 1 : 0 : 0
274+
count = var.aws_alb_redirect_enable && var.aws_certificate_enabled && !contains(local.alb_listen_port, 443) ? 1 : 0 #!contains(local.alb_listen_port, 443) ? 1 : 0 : 0
275275
load_balancer_arn = aws_lb.vm_alb.arn
276276
port = "443"
277277
protocol = "HTTPS"

0 commit comments

Comments
 (0)