prosody / prosody-docker

Docker image building system for the Prosody XMPP server
https://hub.docker.com/r/prosody/prosody/
146 stars 49 forks source link

Fix mod_posix error if prosody user ID was altered #67

Closed selurvedu closed 3 years ago

selurvedu commented 3 years ago

This fixes the following error that occurs if /var/lib/prosody is not owned by UID 101.

mod_posix           error  Couldn't write pidfile at /var/run/prosody/prosody.pid; /var/run/prosody/prosody.pid: Permission denied

Maybe #63 handles permissions better, but I didn't evaluate it properly. I only saw it introduces more significant changes and it's not merged yet, while my MR doesn't really change anything. It simply fixes what #62 missed.

Zash commented 3 years ago

Why do you need a pidfile in Docker?

selurvedu commented 3 years ago

@Zash I don't. Try it yourself with the default config, it is enabled by default.

selurvedu commented 3 years ago

Also, I guess prosodyctl reload may be useful in Docker too.

Zash commented 3 years ago

YOLO-merged. Thanks.