Skip to content

Commit cdca106

Browse files
turegjorupclaude
andcommitted
5402: Update dev compose for split FPM_SERVICE/FPM_PORT
After PR #409 unified the dev and prod nginx templates, the dev compose stack also consumes the production template — but it still passed NGINX_FPM_SERVICE as a combined 'host:port' string. With the template change in this PR ('server ${NGINX_FPM_SERVICE}:${NGINX_FPM_PORT};'), that produced a malformed 'display-phpfpm-1:9000:9000' upstream and nginx failed to start, breaking the Playwright CI run. Split the value in both docker-compose.yml and docker-compose.server.yml. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent c51d2a1 commit cdca106

2 files changed

Lines changed: 4 additions & 2 deletions

File tree

docker-compose.server.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,8 @@ services:
3333
- ./infrastructure/nginx/etc/nginx.conf:/etc/nginx/nginx.conf:ro
3434
- .:/app
3535
environment:
36-
NGINX_FPM_SERVICE: ${COMPOSE_PROJECT_NAME}-phpfpm-1:9000
36+
NGINX_FPM_SERVICE: ${COMPOSE_PROJECT_NAME}-phpfpm-1
37+
NGINX_FPM_PORT: 9000
3738
NGINX_WEB_ROOT: /app/public
3839
NGINX_PORT: 8080
3940
NGINX_MAX_BODY_SIZE: 5M

docker-compose.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,8 @@ services:
6161
- ./infrastructure/nginx/etc/nginx.conf:/etc/nginx/nginx.conf:ro
6262
- .:/app
6363
environment:
64-
NGINX_FPM_SERVICE: ${COMPOSE_PROJECT_NAME}-phpfpm-1:9000
64+
NGINX_FPM_SERVICE: ${COMPOSE_PROJECT_NAME}-phpfpm-1
65+
NGINX_FPM_PORT: 9000
6566
NGINX_WEB_ROOT: /app/public
6667
NGINX_PORT: 8080
6768
NGINX_MAX_BODY_SIZE: 5M

0 commit comments

Comments
 (0)