lucaslorentz / caddy-docker-proxy

Caddy as a reverse proxy for Docker
MIT License
3.06k stars 174 forks source link

More default values including standard Caddyfile #657

Open Ceddicedced opened 2 months ago

Ceddicedced commented 2 months ago

Hi,

i think it would be cool if every entry of the ARGs presented in the Caddy CLI section of the README, would provide a default value. Like some entries e.g. "--scan-stopped-containers" is providing a default value while "--docker-apis-version" is not. It looks a little bit inconsistent. Maybe even placeholders like dafault "null" or "" would be helpfull.

Also while I was setting this up I kind of expected that the standard caddyfile under "/etc/caddy/Caddyfile" would be read, like in the dafault caddy package. It is nice that I can set this via the ENV to enable reading this file, but maybe this should be the default behavior, maybe also with an additional check if that file is empty. But maybe thats just some user eperience fact.