jellyfin / jellyfin-packaging

Jellyfin Packaging and Build Workflows
GNU General Public License v3.0
16 stars 16 forks source link

[Issue]: jellyfin apt upgrade changes permissions of /etc/jellyfin to jellyfin:adm #32

Closed WolfganP closed 1 month ago

WolfganP commented 2 months ago

Please describe your bug

Due a previous media server installation in my debian system, I changed the user:group under which jellyfin runs (even changing the systemd override service conf user & group), and everything went well since original installation. But lately every time I update the server via apt, the folder /etc/jellyfin and its content is changed to ownerjellyfin:adm, and I can't find what I need to modify to comply with my installation and not changed the ownership back to my selected user:group

Reproduction Steps

Update via: sudo apt upgrade

Jellyfin Version

10.9.0

if other:

10.9.7.0

Environment

- OS: Debian 12
- Linux Kernel: 6.1.21
- Virtualization: none
- Clients: android tvbox
- Browser: firefox
- FFmpeg Version:
- Playback Method: direct play
- Hardware Acceleration: none
- GPU Model:
- Plugins:
- Reverse Proxy:
- Base URL:
- Networking:
- Storage:

Jellyfin logs

-

FFmpeg logs

No response

Please attach any browser or client logs here

No response

Please attach any screenshots here

No response

Code of Conduct

joshuaboniface commented 1 month ago

This is being done in the postinst configuration script, but has always been like this. That said it can probably be improved to only run a single time during initial install (i.e. if the dir doesn't exist).

WolfganP commented 1 month ago

This is being done in the postinst configuration script, but has always been like this. That said it can probably be improved to only run a single time during initial install (i.e. if the dir doesn't exist).

Thanks, that gave me the clue to look at the proper config file and modify the user in there. I'll report back if there's any issues in the next upgrade.

And thanks for the latest changes, I saw that the previous version of postinst was assuming adm as default group, now is dynamic