Skip to content

Commit

Permalink
chore: remove legacy TLS ALB listener (#1002)
Browse files Browse the repository at this point in the history
Remove the port 4444 legacy TLS load balancer listener and
its associated security group ingress rule.
  • Loading branch information
patheard authored Nov 2, 2023
1 parent a47978a commit 8e5d35c
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 28 deletions.
21 changes: 0 additions & 21 deletions aws/eks/alb.tf
Original file line number Diff line number Diff line change
Expand Up @@ -66,27 +66,6 @@ resource "aws_lb_listener" "notification-canada-ca-80" {
}
}

# An HTTPS listener with an old SSL policy
# for some clients that cannot upgrade to TLSv1.2
resource "aws_lb_listener" "notification-canada-ca-legacy-tls" {
load_balancer_arn = aws_alb.notification-canada-ca.id
port = 4444
protocol = "HTTPS"
certificate_arn = aws_acm_certificate.notification-canada-ca.arn
#tfsec:ignore:AWS010 Outdated SSL policy
ssl_policy = "ELBSecurityPolicy-2016-08"

default_action {
type = "fixed-response"

fixed_response {
content_type = "text/plain"
message_body = "Forbidden"
status_code = "403"
}
}
}

###
# Document API Specific routing
###
Expand Down
7 changes: 0 additions & 7 deletions aws/eks/securitygroups.tf
Original file line number Diff line number Diff line change
Expand Up @@ -21,13 +21,6 @@ resource "aws_security_group" "notification-canada-ca-alb" {
cidr_blocks = ["0.0.0.0/0"] #tfsec:ignore:AWS008
}

ingress {
protocol = "tcp"
from_port = 4444
to_port = 4444
cidr_blocks = ["0.0.0.0/0"] #tfsec:ignore:AWS008
}

tags = {
CostCenter = "notification-canada-ca-${var.env}"
}
Expand Down

0 comments on commit 8e5d35c

Please sign in to comment.