Skip to content

Fix: Respect custom download and watch directories in settings.json - #329

Open
hvmonteiro wants to merge 1 commit into
linuxserver:masterfrom
hvmonteiro:master
Open

hvmonteiro wants to merge 1 commit into
linuxserver:masterfrom
hvmonteiro:master

Conversation

@hvmonteiro

Copy link
Copy Markdown

Description

This PR fixes hardcoded directory paths in the container initialization script.

Previously, ownership handling assumed that download and watch directories were always located at /downloads and /watch, ignoring custom directory paths configured in settings.json.

Changes

  • Read download-dir, incomplete-dir, and watch-dir dynamically using jq.
  • Remove hardcoded directory paths from ownership checks.
  • Avoid duplicate directory processing.
  • Skip directories that do not exist.
  • Preserve existing non-root-user handling.

Expected behavior

Users configuring custom download, incomplete-download, or watch directories should have those paths respected during container initialization.

Previously, ownership handling assumed that download and watch directories were always located at /downloads and /watch, ignoring custom directory paths configured in settings.json.
@github-actions

Copy link
Copy Markdown

Thanks for opening this pull request! Be sure to follow the pull request template!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants