#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
  • Type-Defect
  • Milestone-0.10
  1. 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#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.

  2. Zash on

    We can look at this when we get to 0.10 packaging

    Changes
    • tags Milestone-0.10 Status-Accepted
    • owner Zash
  3. Zash on

    Changes
    • tags Status-Started
  4. Zash on

    https://hg.prosody.im/debian/rev/0ba4529d28e5

    Changes
    • tags Status-Fixed

New comment