search

wemake-services / caddy-gen

Automated Caddy reverse proxy for docker containers
https://hub.docker.com/r/wemakeservices/caddy-gen
MIT License
250 stars 24 forks source link

fix: Caddy2 doesn't respond to USR1 signal. Fixes #40. #49

Closed couetilc closed 3 years ago

couetilc commented 3 years ago

fixes #40, caddy should reload because it is being run with the --watch flag and docker-gen is writing to /etc/caddy/Caddyfile

Siemko commented 3 years ago

Awesome work! Thank you! @sobolevn I believe this one can be merged right away.

sobolevn commented 3 years ago

Awesome! Thanks!

@couetilc you have contributed a lot of useful stuff already, but can you please also take a look at our CI?

$ curl 127.0.0.1:80
curl: (56) Recv failure: Connection reset by peer
The command "curl 127.0.0.1:80" exited with 56.
couetilc commented 3 years ago

Sure thing, @sobolevn should be fixed in https://github.com/wemake-services/caddy-gen/pull/51

sobolevn commented 3 years ago

Let's test if the CI now works! 👍