search

linuxserver / docker-jellyfin

GNU General Public License v3.0
600 stars 92 forks source link

[FEAT] Make the web files be owned by PUID / PGID #253

Open Ulrar opened 1 month ago

Ulrar commented 1 month ago

Is this a new feature request?

Wanted change

I see this was already requested in https://github.com/linuxserver/docker-jellyfin/issues/182 but closed for inactivity.

I'd be great to chown the /usr/share/jellyfin/web/ folder to abc:abc, so that plugins like intro-skipper and others that require adding their own files to that folder can be installed. I've tested it, and chown abc:abc -R /usr/share/jellyfin/web/ does the trick.

Reason for change

I'm using https://github.com/jumoog/intro-skipper which needs to be able to make changes to that folder to work

Proposed code change

In the dockerfile: 

RUN chown abc:abc -R /usr/share/jellyfin/web/
thespad commented 1 month ago

We try and avoid chowning the container filesystem, especially large directories, because of an overlayfs big that can cause extremely slow chown performance on some devices. We can't chown it at build time because we don't know the uid/gid of the ABC user at that point.

You can use our customisation logic to perform the chown in your container

LinuxServer-CI commented 4 days ago

This issue has been automatically marked as stale because it has not had recent activity. This might be due to missing feedback from OP. It will be closed if no further activity occurs. Thank you for your contributions.