thomas/metube
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Simplify directory ownership changes in entrypoint

Browse Source 
Refactor ownership change logic for directories.
This commit is contained in:
Lachlan Wisdom
2026-01-13 22:47:56 +11:00
committed by GitHub
parent 38a77d19f5
commit 6b9461c8a8
1 changed files with 3 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
docker-entrypoint.sh
View File

@@ -2,12 +2,6 @@
echo "Setting umask to ${UMASK}"
umask ${UMASK}
DOWNLOAD_DIR_CREATED=false
if [ ! -d "${DOWNLOAD_DIR}" ]; then
DOWNLOAD_DIR_CREATED=true
fi
echo "Creating download directory (${DOWNLOAD_DIR}), state directory (${STATE_DIR}), and temp dir (${TEMP_DIR})"
mkdir -p "${DOWNLOAD_DIR}" "${STATE_DIR}" "${TEMP_DIR}"
@@ -15,15 +9,10 @@ if [ `id -u` -eq 0 ] && [ `id -g` -eq 0 ]; then
if [ "${UID}" -eq 0 ]; then
echo "Warning: it is not recommended to run as root user, please check your setting of the UID environment variable"
fi
echo "Changing ownership of state directories to ${UID}:${GID}"
chown -R "${UID}":"${GID}" /app "${STATE_DIR}" "${TEMP_DIR}"
if [ "${CHOWN_DOWNLOAD_DIR:-true}" != "false" ] || [ "${DOWNLOAD_DIR_CREATED}" = "true" ]; then
echo "Changing ownership of download directory (${DOWNLOAD_DIR})"
chown -R "${UID}:${GID}" "${DOWNLOAD_DIR}"
if [ "${CHOWN_DIRS:-true}" != "false" ]; then
echo "Changing ownership of download and state directories to ${UID}:${GID}"
chown -R "${UID}":"${GID}" /app "${DOWNLOAD_DIR}" "${STATE_DIR}" "${TEMP_DIR}"
fi
echo "Running MeTube as user ${UID}:${GID}"
exec su-exec "${UID}":"${GID}" python3 app/main.py
else