add --init to suggested docker run invocations

ergochat / ergo

A modern IRC server (daemon/ircd) written in Go.

https://ergo.chat/

MIT License

2.21k stars 176 forks source link

add --init to suggested docker run invocations #2097

Closed slingamn closed 7 months ago

slingamn commented 8 months ago

tagging @csmith as our resident Docker expert :-p

It seems like this fixes #2096 ( unreaped zombies when using an auth-script or ip-check-script that spawns its own subprocesses, then exits before reaping them), and has basically no downside. Thoughts?

csmith commented 8 months ago

Yeah, this seems like a sensible fix for poorly behaved subprocesses :)

Might be worth adding a small note about why it's needed in the docs? It's a little esoteric and not entirely obvious why ergo would need it at first glance.