#810 Debian packages install /etc/prosody/prosody.cfg.lua with executable bit
Reporter
av6
Owner
Zash
Created
Updated
Stars
★ (1)
Tags
Priority-Medium
Status-Fixed
Milestone-0.10
Type-Defect
av6
on
What steps will reproduce the problem?
1. Add your repo as per https://prosody.im/download/package_repository#debianubuntu
2. Install e.g. prosody-0.10
3. ls -l /etc/prosody/prosody.cfg.lua
Please provide any additional information below.
https://hg.prosody.im/debian/file/tip/prosody.postinst#l47https://hg.prosody.im/debian/rev/9922952c76bb
Debian's own downstream package installs the file as root:prosody 640, but this prevents unprivileged accounts from reading it. I guess you want it readable for all, but it shouldn't require executable bit set, because it's a config file. So instead of 755 I suggest 644, which also works (checked here) for unprivileged accounts trying to run prosodyctl check, for example.
What steps will reproduce the problem? 1. Add your repo as per https://prosody.im/download/package_repository#debianubuntu 2. Install e.g. prosody-0.10 3. ls -l /etc/prosody/prosody.cfg.lua Please provide any additional information below. https://hg.prosody.im/debian/file/tip/prosody.postinst#l47 https://hg.prosody.im/debian/rev/9922952c76bb Debian's own downstream package installs the file as root:prosody 640, but this prevents unprivileged accounts from reading it. I guess you want it readable for all, but it shouldn't require executable bit set, because it's a config file. So instead of 755 I suggest 644, which also works (checked here) for unprivileged accounts trying to run prosodyctl check, for example.
We can look at this when we get to 0.10 packaging
Changeshttps://hg.prosody.im/debian/rev/0ba4529d28e5
Changes