search

jmbannon / ytdl-sub

Lightweight tool to automate downloading and metadata generation with yt-dlp
https://ytdl-sub.readthedocs.io
GNU General Public License v3.0
1.62k stars 62 forks source link

File ownership #1020

Open glottisfaun0000 opened 2 weeks ago

glottisfaun0000 commented 2 weeks ago

Files created don't follow the PID/GID/UMASK of my compose and instead have these permissions: drwxr-xr-x 3 0 0 4096 (root) which means Jellyfin can't delete them from the UI. Is this intended behavior? What's the point of created files not following the env variables?

jmbannon commented 2 weeks ago

Most likely you ran ytdl-sub as root

glottisfaun0000 commented 2 weeks ago

Totally correct, I thought a docker exec would inherit the PUID/PGID from environmental variables on a running container. Adding -u 1000:1000 to my exec fixed it.

With the subscription run working now, the files in Jellyfin are basically unusable because the Season / episode - season.episode string leading the title is so long that the video title information is basically lost. Like for this video downloaded through a subscription, all Jellyfin will show me except as a banner once the video is playing is: Season 2024 - 62901. s2024.e062901 - Game Boy vs. Super Game Boy...

What is the best way to improve this? My subscriptions.yaml now contains under __preset__,overrides, episode_title: "{title}" but that doesn't seem to change anything.

jmbannon commented 2 weeks ago

Jellyfin metadata is read through NFOs, not filename. Make sure your JF library prioritizes local metadata and it should scrape as intended

glottisfaun0000 commented 2 weeks ago

It's currently not possible to disable .nfo metadata. Local metadata will always be fetched and has priority over remote metadata providers like TMDb.

jmbannon commented 2 weeks ago

Can you upload your config, subscriptions, and a screenshot of your JF library settings