From 4cb83ab606566b53700fdee936db18c5c1a5af6e Mon Sep 17 00:00:00 2001 From: JH Date: Mon, 18 Mar 2024 16:22:22 +0100 Subject: [PATCH 1/2] build(compose.yaml): allow overriding TRUSTED_HOSTS and MERCURE_PUBLIC_URL variables Fix issue with invalid TRUSTED_HOSTS and MERCURE_PUBLIC_URL when using http://localhost or :80 as SERVER_NAME --- compose.yaml | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/compose.yaml b/compose.yaml index 9d9b569bc0..ff88e1d4ed 100644 --- a/compose.yaml +++ b/compose.yaml @@ -10,10 +10,10 @@ services: MERCURE_PUBLISHER_JWT_KEY: ${CADDY_MERCURE_JWT_SECRET:-!ChangeThisMercureHubJWTSecretKey!} MERCURE_SUBSCRIBER_JWT_KEY: ${CADDY_MERCURE_JWT_SECRET:-!ChangeThisMercureHubJWTSecretKey!} TRUSTED_PROXIES: ${TRUSTED_PROXIES:-127.0.0.0/8,10.0.0.0/8,172.16.0.0/12,192.168.0.0/16} - TRUSTED_HOSTS: ^${SERVER_NAME:-example\.com|localhost}|php$$ + TRUSTED_HOSTS: ${TRUSTED_HOSTS:-^${SERVER_NAME:-example\.com|localhost}|php$$} DATABASE_URL: postgresql://${POSTGRES_USER:-app}:${POSTGRES_PASSWORD:-!ChangeMe!}@database:5432/${POSTGRES_DB:-app}?serverVersion=${POSTGRES_VERSION:-16}&charset=${POSTGRES_CHARSET:-utf8} MERCURE_URL: ${CADDY_MERCURE_URL:-http://php/.well-known/mercure} - MERCURE_PUBLIC_URL: https://${SERVER_NAME:-localhost}/.well-known/mercure + MERCURE_PUBLIC_URL: ${CADDY_MERCURE_PUBLIC_URL:-https://${SERVER_NAME:-localhost}/.well-known/mercure} MERCURE_JWT_SECRET: ${CADDY_MERCURE_JWT_SECRET:-!ChangeThisMercureHubJWTSecretKey!} volumes: - caddy_data:/data @@ -37,7 +37,7 @@ services: environment: NEXT_PUBLIC_ENTRYPOINT: http://php -###> doctrine/doctrine-bundle ### + ###> doctrine/doctrine-bundle ### database: image: postgres:${POSTGRES_VERSION:-16}-alpine environment: @@ -58,7 +58,7 @@ services: volumes: caddy_data: caddy_config: -###> doctrine/doctrine-bundle ### + ###> doctrine/doctrine-bundle ### db_data: ###< doctrine/doctrine-bundle ### ###> symfony/mercure-bundle ### From 5ce07dd97d98f71c6aae5cb48e78c951dd3b0e09 Mon Sep 17 00:00:00 2001 From: Vincent <407859+vincentchalamon@users.noreply.github.com> Date: Mon, 18 Mar 2024 17:03:41 +0100 Subject: [PATCH 2/2] Apply suggestions from code review --- compose.yaml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/compose.yaml b/compose.yaml index ff88e1d4ed..306a327fd0 100644 --- a/compose.yaml +++ b/compose.yaml @@ -37,7 +37,7 @@ services: environment: NEXT_PUBLIC_ENTRYPOINT: http://php - ###> doctrine/doctrine-bundle ### +###> doctrine/doctrine-bundle ### database: image: postgres:${POSTGRES_VERSION:-16}-alpine environment: @@ -58,7 +58,7 @@ services: volumes: caddy_data: caddy_config: - ###> doctrine/doctrine-bundle ### +###> doctrine/doctrine-bundle ### db_data: ###< doctrine/doctrine-bundle ### ###> symfony/mercure-bundle ###