Volumio2 icon indicating copy to clipboard operation
Volumio2 copied to clipboard

unix user's password cannot be changed without breaking spotify upgrades

Open bablat opened this issue 8 years ago • 4 comments

I believe Volumio should be able to withstand changing the default username/password (volumio:volumio). Requiring a default password to remain unchanged is a very bad security practice and could easily allow an attacker to gain full control to any volumio device on any network.

When upgrading plugins (I use spotify), the process fails if the password is changed. When I change it back to the default the process always completes successfully.

Thanks for building this great software, hope we can get proper snapcast integration (for spotify and airplay as well!).

bablat avatar Jan 10 '17 23:01 bablat

Yes, I definetely agree, its a bad design practice. What we can do is make the plugin installation work when pw is changed, but we need to keep the user volumio

volumio avatar Jan 11 '17 06:01 volumio

I believe that a straightforward temporary fix could be to allow running the plugin installer script under sudo with the NOPASSWD directive (e.g. http://unix.stackexchange.com/questions/18830/how-to-run-a-specific-program-as-root-without-a-password-prompt). This would simply remove the prompt (leaving a risk there, easy escalation from volumio->root if that script is compromised) but far better than requiring password to remain unchanged. Another option would be to prompt for password if the sudo command fails, and then retry it.

Not sure if the plugin installer even needs root access to run? I assume it's not critical for most plugins but you wanted to keep the possibility available for future plugins?

bablat avatar Jan 11 '17 10:01 bablat

I also believe that once this is addressed, volumio should prompt for a new password once installed. default passwords are bad even if a user is allowed to change them. I'd hate for volumio to get the wrong type of publicity: http://www.welivesecurity.com/2016/10/24/10-things-know-october-21-iot-ddos-attacks/

bablat avatar Jan 11 '17 10:01 bablat

Could the "volumio"-user be interpreted as a "system"-user and adding a "admin" user which will be prompted for a password during setup/configuration?

gonace avatar Mar 05 '17 10:03 gonace