Skip to content

Commit

Permalink
Run fixtures on non-privileged port
Browse files Browse the repository at this point in the history
But still expose it as 80 through the load balancer

#patch
  • Loading branch information
gregtyler committed Oct 15, 2024
1 parent df58597 commit 15166e4
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 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

0 comments on commit 15166e4

Please sign in to comment.