Skip to content

Commit

Permalink
fix: move haproxy stats ports back onto nonroot range internally
Browse files Browse the repository at this point in the history
  • Loading branch information
gilesw committed Feb 19, 2025
1 parent d089d75 commit 56ecb59
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 5 deletions.
4 changes: 2 additions & 2 deletions docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ services:
- orcid-haprouter/default.env
- orcid-haprouter/deployment.env
ports:
- 0.0.0.0:678:789 # stats
- 0.0.0.0:678:1936 # stats
- 0.0.0.0:7983:7983 # solr read
- 0.0.0.0:6983:6983 # solr write
- 0.0.0.0:7432:7432 # postgres read
Expand All @@ -67,7 +67,7 @@ services:
context: .
dockerfile: orcid-lb/Dockerfile
ports:
- 0.0.0.0:891:789 # stats
- 0.0.0.0:891:1936 # stats
- 0.0.0.0:80:80
- 0.0.0.0:443:443
networks:
Expand Down
3 changes: 2 additions & 1 deletion orcid-haprouter/haproxy.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,8 @@ defaults
stats show-modules
stats show-legends

listen stats-789
# haproxy in docker runs as non root so even admin ports must be > 1024
listen stats-1936
description haproute Loadbalancer
bind 0.0.0.0:789
mode http
Expand Down
5 changes: 3 additions & 2 deletions orcid-lb/haproxy.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -34,9 +34,10 @@ defaults
stats show-modules
stats show-legends

listen stats-789
# haproxy in docker runs as non root so even admin ports must be > 1024
listen stats-1936
description haproute Loadbalancer
bind 0.0.0.0:789
bind 0.0.0.0:1936
mode http
stats enable
stats uri /
Expand Down

0 comments on commit 56ecb59

Please sign in to comment.