[Documentation] Wrong environment variables syntax in README for Portainer

[skylanix]

Hello guys, I love your project! To help improve it, here's a correction for the README.

When using the "Advanced mode" in Portainer to easily paste the environment variables, the current syntax doesn't work. The README shows colons : but it should use equals signs = instead.

Current syntax (doesn't work):

VIDEOS_FOLDER_DOCKER_HOST:/mnt/data/videos
TMP_DOWNLOAD_FOLDER_DOCKER_HOST:/mnt/data/hometube/tmp
YOUTUBE_COOKIES_FILE_PATH_DOCKER_HOST:/opt/cookies/youtube.txt

Correct syntax:

VIDEOS_FOLDER_DOCKER_HOST=/mnt/data/videos
TMP_DOWNLOAD_FOLDER_DOCKER_HOST=/mnt/data/hometube/tmp
YOUTUBE_COOKIES_FILE_PATH_DOCKER_HOST=/opt/cookies/youtube.txt
TZ=America/New_York
PORT=8501
COOKIES_FROM_BROWSER=chrome

Additional suggestion: It would also be practical to add these lines in the docker-compose.yml example:

container_name: hometube
ports:
  - "${PORT}:8501"
environment:
  TZ: "${TZ}"
  COOKIES_FROM_BROWSER: "${COOKIES_FROM_BROWSER}"

This would help users quickly.

Thanks for this great project!

[EgalitarianMonkey]

Hi skylanix,
Thanks for pointing this out. You are completely right.
I made some documentation corrections regarding your suggestions.
I didn't let COOKIES_FROM_BROWSER as principal authentication solution as I believe most of the setups using HomeTube do it through YOUTUBE_COOKIES_FILE_PATH.
Tell me if it's better 😉.

[gregbtm]

Using portainer with these env variables, I don't seem to have any luck with the application reading the txt file. I think I may have misconfigured it, but its a little confusing having two files, one titled just youtube.txt and the other youtube_cookies.txt

This is from the portainer instructions -

VIDEOS_FOLDER_DOCKER_HOST=/mnt/data/videos
TMP_DOWNLOAD_FOLDER_DOCKER_HOST=/mnt/data/hometube/tmp
YOUTUBE_COOKIES_FILE_PATH_DOCKER_HOST=/opt/cookies/youtube.txt
TZ=America/New_York
PORT=8501
YOUTUBE_COOKIES_FILE_PATH=/config/youtube_cookies.txt
#COOKIES_FROM_BROWSER=chrome

Also, other question - What does the txt file have to contain - just the header string?

[EgalitarianMonkey]

Hi gregbtm,

I see some issues:

• You don't want to change the YOUTUBE_COOKIES_FILE_PATH=/config/youtube_cookies.txt parameter in the docker context
• VIDEOS_FOLDER_DOCKER_HOST should be under /volume1/docker/hometube/xxx like /volume1/docker/hometube/videos or /volume1/docker/hometube/downloads
• Also make sure there is write right on your cookie file /volume1/docker/hometube/config/youtube.txt

And there is my personal config in Portainer:

Finally, double check your docker mounts.
Tell me if you can fix it.