search

linuxserver / docker-radarr

GNU General Public License v3.0
646 stars 104 forks source link

update radarr application files to correct ownership #87

Closed danaith closed 4 years ago

danaith commented 4 years ago

linuxserver.io

We welcome all PR’s though this doesn’t guarantee it will be accepted.

Description:

Update permissions of /opt so that installed radarr files have correct userid and group if a different PUID and GUID is passed in docker options.

Benefits of this PR and context:

As above

How Has This Been Tested?

thelamer commented 4 years ago

As above huh, can you be a little more specific as to what this fixes ?

danaith commented 4 years ago

As above huh, can you be a little more specific as to what this fixes ?

Sorry, thought it was pretty self explanatory, it fixes permissions on the radarr application directory so that radarr stops reporting that there are incorrect permissions on its home directory preventing the in built auto updates in radarr. The default uid is set to 1001 but if you use a PUID of anything other then 1001 on the container you will see the error in radarr on the system tab.

thelamer commented 4 years ago

We don't want people updating in the container they should use docker to update , long startup chowns were specifically removed because of this. If that ever changes or the application needs write permissions for some functionality/caching we will implement a symlink or chowning .