Opened 8 years ago
Last modified 8 years ago
#5812 new Bug
Set umask only if explicitly specified
Reported by: | mike.dld | Owned by: | jordan |
---|---|---|---|
Priority: | Normal | Milestone: | None Set |
Component: | libtransmission | Version: | 2.84 |
Severity: | Normal | Keywords: | |
Cc: |
Description
Although #2234 was about daemon only, ticket component got changed to libtransmission and changes made affected all the clients. However, GUI clients should most probably inherit their umask. Attached patch makes "umask" parameter in settings.json optional, and if not present umask won't be modified and parameter won't be added to configuration file on save. The change won't affect existing installations, where "umask" is already present in settings.json, but for new installations users would need to add the parameter explicitly if they want to change it from default (inherited) one.
Attachments (1)
Note: See
TracTickets for help on using
tickets.
On a second thought, I decided to always save "umask" parameter to settings json and always return it on settings request. I guess Shift web interface (and it may not be the only one) shows all the available parameters, and "umask" won't be there for modification if I skip it. But parameter value of -1 would still mean that umask shouldn't be changed. Patch updated.