From a9490da8a813ff02481fca6dc57beb7ecad30cfb Mon Sep 17 00:00:00 2001 From: Claude Date: Fri, 20 Mar 2026 12:49:39 +0000 Subject: [PATCH] Fix Caddy startup: remove empty ACME_EMAIL that caused parse error MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Caddy's email directive requires a non-empty argument. Since ACME_EMAIL wasn't set, Caddy failed to parse the config. Email is optional for Let's Encrypt — remove the directive entirely and document it as a manual opt-in comment. --- infra/.env.example | 4 ++-- infra/docker-compose.yml | 1 - proxy/Caddyfile | 4 ++-- 3 files changed, 4 insertions(+), 5 deletions(-) diff --git a/infra/.env.example b/infra/.env.example index 8ff5295..9f654ef 100644 --- a/infra/.env.example +++ b/infra/.env.example @@ -1,5 +1,5 @@ # Your domain — apps will be served at .yourdomain.com DOMAIN_SUFFIX=yourdomain.com -# Email address for Let's Encrypt expiry notices (required for HTTPS) -ACME_EMAIL=you@example.com +# Optional: email for Let's Encrypt expiry notices. +# If you want this, uncomment the `email` line in proxy/Caddyfile instead. diff --git a/infra/docker-compose.yml b/infra/docker-compose.yml index c041ec8..8bc6369 100644 --- a/infra/docker-compose.yml +++ b/infra/docker-compose.yml @@ -55,7 +55,6 @@ services: - "2019:2019" # admin API environment: DOMAIN_SUFFIX: ${DOMAIN_SUFFIX:-localhost} - ACME_EMAIL: ${ACME_EMAIL:-} volumes: - ../proxy/Caddyfile:/etc/caddy/Caddyfile:ro - caddy-data:/data diff --git a/proxy/Caddyfile b/proxy/Caddyfile index 322af1c..2cf9439 100644 --- a/proxy/Caddyfile +++ b/proxy/Caddyfile @@ -15,8 +15,8 @@ # Admin API — used by hiy-server to add/remove app routes dynamically. admin 0.0.0.0:2019 - # Contact email for Let's Encrypt expiry notices. - email {$ACME_EMAIL} + # Uncomment and set your email for Let's Encrypt expiry notices: + # email you@example.com } # HIY dashboard — served at your root domain.