From d1574448774c1350af850785eff93d9577dfc021 Mon Sep 17 00:00:00 2001 From: Andrew Keeton <208120+akeeton@users.noreply.github.com> Date: Thu, 11 Jun 2026 17:21:56 -0400 Subject: [PATCH 1/2] Create AUDIO_DOWNLOAD_DIR in Docker image --- Dockerfile | 1 + docker-entrypoint.sh | 6 +++--- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/Dockerfile b/Dockerfile index f9896bc..5a25570 100644 --- a/Dockerfile +++ b/Dockerfile @@ -61,6 +61,7 @@ ENV PGID=1000 ENV UMASK=022 ENV DOWNLOAD_DIR=/downloads +ENV AUDIO_DOWNLOAD_DIR=/downloads ENV STATE_DIR=/downloads/.metube ENV TEMP_DIR=/downloads ENV PORT=8081 diff --git a/docker-entrypoint.sh b/docker-entrypoint.sh index e1d976c..615d477 100644 --- a/docker-entrypoint.sh +++ b/docker-entrypoint.sh @@ -5,8 +5,8 @@ PGID="${GID:-$PGID}" echo "Setting umask to ${UMASK}" umask ${UMASK} -echo "Creating download directory (${DOWNLOAD_DIR}), state directory (${STATE_DIR}), and temp dir (${TEMP_DIR})" -mkdir -p "${DOWNLOAD_DIR}" "${STATE_DIR}" "${TEMP_DIR}" +echo "Creating download directory (${DOWNLOAD_DIR}), audio download directory (${AUDIO_DOWNLOAD_DIR}), state directory (${STATE_DIR}), and temp dir (${TEMP_DIR})" +mkdir -p "${DOWNLOAD_DIR}" "${AUDIO_DOWNLOAD_DIR}" "${STATE_DIR}" "${TEMP_DIR}" do_upgrade() { echo "Upgrading yt-dlp to nightly channel..." @@ -56,7 +56,7 @@ if [ `id -u` -eq 0 ] && [ `id -g` -eq 0 ]; then fi if [ "${CHOWN_DIRS:-true}" != "false" ]; then echo "Changing ownership of download and state directories to ${PUID}:${PGID}" - chown -R "${PUID}":"${PGID}" /app "${DOWNLOAD_DIR}" "${STATE_DIR}" "${TEMP_DIR}" + chown -R "${PUID}":"${PGID}" /app "${DOWNLOAD_DIR}" "${AUDIO_DOWNLOAD_DIR}" "${STATE_DIR}" "${TEMP_DIR}" fi if nightly_enabled; then echo "YTDL_NIGHTLY_UPDATE_TIME is set to ${YTDL_NIGHTLY_UPDATE_TIME}; upgrading yt-dlp on startup" From 04b93667647fe11a28a4b99734da508a1fc948fb Mon Sep 17 00:00:00 2001 From: Andrew Keeton <208120+akeeton@users.noreply.github.com> Date: Wed, 17 Jun 2026 17:17:44 -0400 Subject: [PATCH 2/2] Incorporate PR feedback Move the default assignment of AUDIO_DOWNLOAD_DIR from the Dockerfile to docker-entrypoint.sh, and change the default value from "/downloads" to $DOWNLOAD_DIR. --- Dockerfile | 1 - docker-entrypoint.sh | 1 + 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/Dockerfile b/Dockerfile index 5a25570..f9896bc 100644 --- a/Dockerfile +++ b/Dockerfile @@ -61,7 +61,6 @@ ENV PGID=1000 ENV UMASK=022 ENV DOWNLOAD_DIR=/downloads -ENV AUDIO_DOWNLOAD_DIR=/downloads ENV STATE_DIR=/downloads/.metube ENV TEMP_DIR=/downloads ENV PORT=8081 diff --git a/docker-entrypoint.sh b/docker-entrypoint.sh index 615d477..ad74e2f 100644 --- a/docker-entrypoint.sh +++ b/docker-entrypoint.sh @@ -2,6 +2,7 @@ PUID="${UID:-$PUID}" PGID="${GID:-$PGID}" +AUDIO_DOWNLOAD_DIR="${AUDIO_DOWNLOAD_DIR:-$DOWNLOAD_DIR}" echo "Setting umask to ${UMASK}" umask ${UMASK}