ashpieboop / aur-caddy

https://aur.archlinux.org/packages/caddy2
1 stars 1 forks source link

Adding optional EnvironmentFile #10

Closed balki closed 3 years ago

balki commented 3 years ago

Reference: https://caddyserver.com/docs/caddyfile/concepts#environment-variables https://www.freedesktop.org/software/systemd/man/systemd.exec.html#EnvironmentFile=

ashpieboop commented 3 years ago

Thanks a lot for this amazing pull request! I will merge this asap.

ashpieboop commented 3 years ago

Actually isn't envfile a bit redundant for a file name? I would suggest the more simple /etc/caddy/env (please lmk if you see any objection).

balki commented 3 years ago

Actually isn't envfile a bit redundant for a file name? I would suggest the more simple /etc/caddy/env (please lmk if you see any objection).

Caddy 1.0 automatically read the file named envfile (at least it did in my setup). Either name is fine with me. :)

ashpieboop commented 3 years ago

I don't want to impose my personal opinions on this so let's stick to the traditional envfile. Thanks again!