From 0bf72fd5f061c9c851aa4ae8f055e8d23eae68b3 Mon Sep 17 00:00:00 2001 From: FaySmash <30392780+FaySmash@users.noreply.github.com> Date: Wed, 15 Jan 2025 18:46:05 +0100 Subject: [PATCH 1/3] Update README.md Added a small section about potential permission issues with volumes. --- README.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/README.md b/README.md index a01f9830..e49a30c7 100644 --- a/README.md +++ b/README.md @@ -138,6 +138,8 @@ services: - PGID=1000 ``` +> [!IMPORTANT] +> If the `/downloads` directory is mounted to a volume which points to a remote storage, make sure to suppy the `UID` and `GID` parameters in the driver options, to match the `PUID` and `PGID` specified as environment variables to prevent permission issues. [See this issue for details](https://github.com/meeb/tubesync/issues/616#issuecomment-2593458282) ## Optional authentication From eff92e3469accb46e941f75e969d86eaf89baf17 Mon Sep 17 00:00:00 2001 From: FaySmash <30392780+FaySmash@users.noreply.github.com> Date: Fri, 17 Jan 2025 19:46:10 +0100 Subject: [PATCH 2/3] Update README.md Revised version of the section about potential permission issues with Samba volumes. --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index e49a30c7..d695221f 100644 --- a/README.md +++ b/README.md @@ -139,7 +139,7 @@ services: ``` > [!IMPORTANT] -> If the `/downloads` directory is mounted to a volume which points to a remote storage, make sure to suppy the `UID` and `GID` parameters in the driver options, to match the `PUID` and `PGID` specified as environment variables to prevent permission issues. [See this issue for details](https://github.com/meeb/tubesync/issues/616#issuecomment-2593458282) +> If the `/downloads` directory is mounted to a [Samba volume](https://docs.docker.com/engine/storage/volumes/#create-cifssamba-volumes), make sure to suppy the `UID` and `GID` parameters in the driver options. These have to be the same as the `PUID` and `PGID`, which were specified as environment variables. This prevents issues when executing file actions (like writing metadata). [See this issue for details](https://github.com/meeb/tubesync/issues/616#issuecomment-2593458282) ## Optional authentication From c5cdbe4a550fc5f38e7cd0533f7ad28e237f8390 Mon Sep 17 00:00:00 2001 From: FaySmash <30392780+FaySmash@users.noreply.github.com> Date: Sun, 19 Jan 2025 17:18:11 +0100 Subject: [PATCH 3/3] Update README.md Fix the suppy => supply misspelling +alternate phrasing Co-authored-by: tcely --- README.md | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/README.md b/README.md index d695221f..ad437bba 100644 --- a/README.md +++ b/README.md @@ -139,7 +139,10 @@ services: ``` > [!IMPORTANT] -> If the `/downloads` directory is mounted to a [Samba volume](https://docs.docker.com/engine/storage/volumes/#create-cifssamba-volumes), make sure to suppy the `UID` and `GID` parameters in the driver options. These have to be the same as the `PUID` and `PGID`, which were specified as environment variables. This prevents issues when executing file actions (like writing metadata). [See this issue for details](https://github.com/meeb/tubesync/issues/616#issuecomment-2593458282) +> If the `/downloads` directory is mounted from a [Samba volume](https://docs.docker.com/engine/storage/volumes/#create-cifssamba-volumes), be sure to also supply the `uid` and `gid` mount parameters in the driver options. +> These must be matched to the `PUID` and `PGID` values, which were specified as environment variables. +> +> Matching these user and group ID numbers prevents issues when executing file actions, such as writing metadata. See [this issue](https://github.com/meeb/tubesync/issues/616#issuecomment-2593458282) for details. ## Optional authentication