sander/Hostityourself
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix Caddy startup: remove empty ACME_EMAIL that caused parse error

Browse source 
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.
This commit is contained in:
Claude 2026-03-20 12:49:39 +00:00
parent dc59293c5e
commit a9490da8a8
No known key found for this signature in database
3 changed files with 4 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
infra/.env.example
View file

@ -1,5 +1,5 @@
# Your domain — apps will be served at <name>.yourdomain.com # Your domain — apps will be served at <name>.yourdomain.com
DOMAIN_SUFFIX=yourdomain.com DOMAIN_SUFFIX=yourdomain.com
# Email address for Let's Encrypt expiry notices (required for HTTPS) # Optional: email for Let's Encrypt expiry notices.
ACME_EMAIL=you@example.com # If you want this, uncomment the `email` line in proxy/Caddyfile instead.

1
infra/docker-compose.yml
View file

@ -55,7 +55,6 @@ services:
- "2019:2019" # admin API - "2019:2019" # admin API
environment: environment:
DOMAIN_SUFFIX: ${DOMAIN_SUFFIX:-localhost} DOMAIN_SUFFIX: ${DOMAIN_SUFFIX:-localhost}
ACME_EMAIL: ${ACME_EMAIL:-}
volumes: volumes:
- ../proxy/Caddyfile:/etc/caddy/Caddyfile:ro - ../proxy/Caddyfile:/etc/caddy/Caddyfile:ro
- caddy-data:/data - caddy-data:/data

4
proxy/Caddyfile
View file

@ -15,8 +15,8 @@
# Admin API — used by hiy-server to add/remove app routes dynamically. # Admin API — used by hiy-server to add/remove app routes dynamically.
admin 0.0.0.0:2019 admin 0.0.0.0:2019
# Contact email for Let's Encrypt expiry notices. # Uncomment and set your email for Let's Encrypt expiry notices:
email {$ACME_EMAIL} # email you@example.com
} }
# HIY dashboard — served at your root domain. # HIY dashboard — served at your root domain.