From 329e84989449ed25d12a4d66ab7ec2fd84b94e46 Mon Sep 17 00:00:00 2001 From: Jose Diaz-Gonzalez Date: Wed, 13 Dec 2017 01:27:29 -0500 Subject: [PATCH] fix: do not allow shadowing of the CACHE_DIR variable Closes #2901 --- plugins/common/functions | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/plugins/common/functions b/plugins/common/functions index 450c1d7bd..b19546ad9 100755 --- a/plugins/common/functions +++ b/plugins/common/functions @@ -443,7 +443,7 @@ dokku_build() { local cid verify_app_name "$APP" - local CACHE_DIR="$DOKKU_ROOT/$APP/cache" + local DOKKU_APP_CACHE_DIR="$DOKKU_ROOT/$APP/cache" eval "$(config_export app "$APP")" pushd "$TMP_WORK_DIR" &> /dev/null @@ -454,7 +454,7 @@ dokku_build() { cid=$(tar -c . | docker run "$DOKKU_GLOBAL_RUN_ARGS" -i -a stdin "$DOKKU_IMAGE" /bin/bash -c "mkdir -p /app && tar -xC /app") test "$(docker wait "$cid")" -eq 0 docker commit "$cid" "$IMAGE" > /dev/null - [[ -d $CACHE_DIR ]] || mkdir -p "$CACHE_DIR" + [[ -d $DOKKU_APP_CACHE_DIR ]] || mkdir -p "$DOKKU_APP_CACHE_DIR" plugn trigger pre-build-buildpack "$APP" local DOCKER_ARGS=$(: | plugn trigger docker-args-build "$APP" "$IMAGE_SOURCE_TYPE") @@ -462,7 +462,7 @@ dokku_build() { declare -a ARG_ARRAY eval "ARG_ARRAY=($DOCKER_ARGS)" # shellcheck disable=SC2086 - cid=$(docker run $DOKKU_GLOBAL_RUN_ARGS -d -v $CACHE_DIR:/cache -e CACHE_PATH=/cache "${ARG_ARRAY[@]}" $IMAGE /build) + cid=$(docker run $DOKKU_GLOBAL_RUN_ARGS -d -v $DOKKU_APP_CACHE_DIR:/cache -e CACHE_PATH=/cache "${ARG_ARRAY[@]}" $IMAGE /build) docker attach "$cid" test "$(docker wait "$cid")" -eq 0 docker commit "$cid" "$IMAGE" > /dev/null