Skip to content

Commit

Permalink
Run fixtures on non-privileged port
Browse files Browse the repository at this point in the history
But it will still be providing 443 at the load balancer.

#patch
  • Loading branch information
gregtyler committed Oct 15, 2024
1 parent df58597 commit ccbc2ac
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
4 changes: 2 additions & 2 deletions fixtures/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -24,9 +24,9 @@ COPY fixtures/static static
COPY fixtures/templates templates
COPY docs/schemas static/schemas

EXPOSE 80
EXPOSE 8080

RUN addgroup -S app && adduser -S -g app app
USER app

CMD [ "flask", "run", "--host", "0.0.0.0", "--port", "80"]
CMD [ "flask", "run", "--host", "0.0.0.0", "--port", "8080"]
6 changes: 3 additions & 3 deletions terraform/modules/fixtures_service/ecs.tf
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ resource "aws_ecs_service" "fixtures" {
load_balancer {
target_group_arn = aws_lb_target_group.fixtures.arn
container_name = "fixtures"
container_port = 80
container_port = 8080
}

network_configuration {
Expand Down Expand Up @@ -60,8 +60,8 @@ locals {
name = "fixtures",
portMappings = [
{
containerPort = 80,
hostPort = 80,
containerPort = 8080,
hostPort = 8080,
protocol = "tcp"
}
],
Expand Down
2 changes: 1 addition & 1 deletion terraform/modules/fixtures_service/security_group.tf
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ resource "aws_security_group_rule" "alb_ingress" {
type = "ingress"
protocol = "tcp"
from_port = 80
to_port = 80
to_port = 8080
source_security_group_id = aws_security_group.loadbalancer_gov_wifi.id
security_group_id = aws_security_group.ecs.id
description = "Inbound from the ALB"
Expand Down

0 comments on commit ccbc2ac

Please sign in to comment.