diff --git a/ISSUE_TEMPLATE.md b/ISSUE_TEMPLATE.md index a1b729960..03bb487e2 100644 --- a/ISSUE_TEMPLATE.md +++ b/ISSUE_TEMPLATE.md @@ -39,4 +39,4 @@ - If using a `Dockerfile`, the contents of that file - If it exists, the contents of your `Procfile`. - Output of failing Dokku commands after running `dokku trace:on` - (BEWARE: `trace:on` will print environment variables for some commands, be sure you're not exposing any sensitive information when posting issues. You may replace these values with XXXXXXX): + (BEWARE: `trace:on` will print environment variables for some commands, be sure you're not exposing any sensitive information when posting issues. You may replace these values with XXXXXX): diff --git a/contrib/release-dokku b/contrib/release-dokku index f56d4e564..9dd3eb70f 100755 --- a/contrib/release-dokku +++ b/contrib/release-dokku @@ -3,7 +3,7 @@ set -eo pipefail [[ $TRACE ]] && set -x readonly ROOT_DIR="$(cd "$(dirname "$(dirname "${BASH_SOURCE[0]}")")" && pwd)" -readonly TMP_WORK_DIR="$(mktemp -d "/tmp/dokku-release.XXXX")" +readonly TMP_WORK_DIR="$(mktemp -d "/tmp/dokku-release.XXXXXX")" readonly DOKKU_GIT_REV="$(git rev-parse HEAD)" trap 'rm -rf "$TMP_WORK_DIR" >/dev/null' RETURN INT TERM EXIT diff --git a/contrib/release-dokku-update b/contrib/release-dokku-update index a431b0c87..ed1fafb77 100755 --- a/contrib/release-dokku-update +++ b/contrib/release-dokku-update @@ -3,7 +3,7 @@ set -eo pipefail [[ $TRACE ]] && set -x readonly ROOT_DIR="$(cd "$(dirname "$(dirname "${BASH_SOURCE[0]}")")" && pwd)" -readonly TMP_WORK_DIR="$(mktemp -d "/tmp/dokku-release.XXXX")" +readonly TMP_WORK_DIR="$(mktemp -d "/tmp/dokku-release.XXXXXX")" readonly DOKKU_GIT_REV="$(git rev-parse HEAD)" trap 'rm -rf "$TMP_WORK_DIR" >/dev/null' RETURN INT TERM EXIT diff --git a/contrib/release-herokuish b/contrib/release-herokuish index 9293441df..367de3530 100755 --- a/contrib/release-herokuish +++ b/contrib/release-herokuish @@ -3,7 +3,7 @@ set -eo pipefail [[ $TRACE ]] && set -x readonly ROOT_DIR="$(cd "$(dirname "$(dirname "${BASH_SOURCE[0]}")")" && pwd)" -readonly TMP_WORK_DIR="$(mktemp -d "/tmp/dokku-release.XXXX")" +readonly TMP_WORK_DIR="$(mktemp -d "/tmp/dokku-release.XXXXXX")" readonly DOKKU_GIT_REV="$(git rev-parse HEAD)" trap 'rm -rf "$TMP_WORK_DIR" >/dev/null' RETURN INT TERM EXIT diff --git a/contrib/release-plugin b/contrib/release-plugin index f06996d5b..a5ae29467 100755 --- a/contrib/release-plugin +++ b/contrib/release-plugin @@ -2,7 +2,7 @@ set -eo pipefail [[ $TRACE ]] && set -x -readonly TMP_WORK_DIR="$(mktemp -d "/tmp/dokku-plugin-release.XXXX")" +readonly TMP_WORK_DIR="$(mktemp -d "/tmp/dokku-plugin-release.XXXXXX")" trap 'rm -rf "$TMP_WORK_DIR" >/dev/null' RETURN INT TERM EXIT diff --git a/docs/development/plugin-creation.md b/docs/development/plugin-creation.md index f9d2be9ce..642655163 100644 --- a/docs/development/plugin-creation.md +++ b/docs/development/plugin-creation.md @@ -194,7 +194,7 @@ Avoid copying files from running containers as these files may change over time. ```shell source "$PLUGIN_CORE_AVAILABLE_PATH/common/functions" -local TMP_FILE=$(mktemp "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXX") +local TMP_FILE=$(mktemp "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXXXX") trap "rm -rf '$TMP_FILE' >/dev/null" RETURN INT TERM local IMAGE_TAG="$(get_running_image_tag "$APP")" diff --git a/plugins/app-json/internal-functions b/plugins/app-json/internal-functions index 5fed09bce..fe2aca3f7 100755 --- a/plugins/app-json/internal-functions +++ b/plugins/app-json/internal-functions @@ -8,7 +8,7 @@ source "$PLUGIN_AVAILABLE_PATH/ps/functions" get_phase_script() { declare desc="extracts app.json from app image and returns the appropriate json key/value" declare IMAGE_TAG="$1" PHASE_SCRIPT_KEY="$2" - local APP_JSON_FILE=$(mktemp "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXX") + local APP_JSON_FILE=$(mktemp "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXXXX") trap "rm -rf '$APP_JSON_FILE' >/dev/null" RETURN INT TERM copy_from_image "$IMAGE" "app.json" "$APP_JSON_FILE" 2>/dev/null || true diff --git a/plugins/certs/subcommands/add b/plugins/certs/subcommands/add index 2cc7938c8..fcf8425c3 100755 --- a/plugins/certs/subcommands/add +++ b/plugins/certs/subcommands/add @@ -42,7 +42,7 @@ certs_set() { # importing from file true elif is_tar_import; then - local CERTS_SET_TMP_WORK_DIR=$(mktemp -d "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXX") + local CERTS_SET_TMP_WORK_DIR=$(mktemp -d "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXXXX") pushd "$CERTS_SET_TMP_WORK_DIR" &>/dev/null trap "popd &>/dev/null || true; rm -rf '$CERTS_SET_TMP_WORK_DIR' >/dev/null" RETURN tar xvf - <&0 diff --git a/plugins/certs/subcommands/generate b/plugins/certs/subcommands/generate index 9a4e9a0e4..4d691bce3 100755 --- a/plugins/certs/subcommands/generate +++ b/plugins/certs/subcommands/generate @@ -13,7 +13,7 @@ certs_generate_cmd() { local APP_SSL_PATH="$DOKKU_ROOT/$APP/tls" if [[ ! -f "$APP_SSL_PATH/server.key" ]] && [[ ! -f "$APP_SSL_PATH/server.crt" ]]; then - local CERTS_GENERATE_TMP_WORK_DIR=$(mktemp -d "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXX") + local CERTS_GENERATE_TMP_WORK_DIR=$(mktemp -d "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXXXX") pushd "$CERTS_GENERATE_TMP_WORK_DIR" >/dev/null trap "popd &>/dev/null || true; rm -rf '$CERTS_GENERATE_TMP_WORK_DIR' >/dev/null" INT TERM EXIT diff --git a/plugins/common/functions b/plugins/common/functions index 2d6c87f34..ebf714ab8 100755 --- a/plugins/common/functions +++ b/plugins/common/functions @@ -390,7 +390,7 @@ copy_from_image() { fi fi - TMP_FILE_COMMAND_OUTPUT=$(mktemp "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXX") + TMP_FILE_COMMAND_OUTPUT=$(mktemp "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXXXX") trap "rm -rf '$TMP_FILE_COMMAND_OUTPUT' >/dev/null 2>&1 || true" RETURN local CID=$("$DOCKER_BIN" create "${DOCKER_CREATE_LABEL_ARGS[@]}" $DOKKU_GLOBAL_RUN_ARGS "$IMAGE") @@ -1029,7 +1029,7 @@ release_advisory_lock() { suppress_output() { declare desc="suppress all output from a given command unless there is an error" local TMP_COMMAND_OUTPUT - TMP_COMMAND_OUTPUT=$(mktemp "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXX") + TMP_COMMAND_OUTPUT=$(mktemp "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXXXX") trap "rm -rf '$TMP_COMMAND_OUTPUT' >/dev/null" RETURN "$@" >"$TMP_COMMAND_OUTPUT" 2>&1 || { diff --git a/plugins/git/functions b/plugins/git/functions index c31dc5236..c980841aa 100755 --- a/plugins/git/functions +++ b/plugins/git/functions @@ -31,7 +31,7 @@ git_build_app_repo() { verify_app_name "$APP" # clean up after ourselves - local GIT_BUILD_APP_REPO_TMP_WORK_DIR=$(mktemp -d "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXX") + local GIT_BUILD_APP_REPO_TMP_WORK_DIR=$(mktemp -d "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXXXX") trap "rm -rf '$GIT_BUILD_APP_REPO_TMP_WORK_DIR' >/dev/null" RETURN INT TERM EXIT local TMP_TAG="dokku/$REV" diff --git a/plugins/nginx-vhosts/functions b/plugins/nginx-vhosts/functions index ce3642cf6..5319879af 100755 --- a/plugins/nginx-vhosts/functions +++ b/plugins/nginx-vhosts/functions @@ -300,8 +300,8 @@ nginx_build_config() { done local PROXY_UPSTREAM_PORTS="$(echo "$PROXY_UPSTREAM_PORTS" | xargs)" - local NGINX_BUILD_CONFIG_TMP_WORK_DIR=$(mktemp -d "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXX") - local NGINX_CONF=$(mktemp --tmpdir="${NGINX_BUILD_CONFIG_TMP_WORK_DIR}" "nginx.conf.XXXX") + local NGINX_BUILD_CONFIG_TMP_WORK_DIR=$(mktemp -d "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXXXX") + local NGINX_CONF=$(mktemp --tmpdir="${NGINX_BUILD_CONFIG_TMP_WORK_DIR}" "nginx.conf.XXXXXX") local CUSTOM_NGINX_TEMPLATE="$NGINX_BUILD_CONFIG_TMP_WORK_DIR/$NGINX_TEMPLATE_NAME" # shellcheck disable=SC2086 trap "rm -rf '$NGINX_CONF' '$NGINX_BUILD_CONFIG_TMP_WORK_DIR' >/dev/null" RETURN INT TERM EXIT diff --git a/plugins/nginx-vhosts/internal-functions b/plugins/nginx-vhosts/internal-functions index dac6f6a5d..576db3504 100755 --- a/plugins/nginx-vhosts/internal-functions +++ b/plugins/nginx-vhosts/internal-functions @@ -27,7 +27,7 @@ nginx_vhosts_is_valid_nginx_config_func() { declare desc="checks if an app has a valid nginx config" declare APP="$1" local VALIDATE_TEMPLATE="$PLUGIN_AVAILABLE_PATH/nginx-vhosts/templates/validate.conf.sigil" - local TMP_OUTPUT=$(mktemp "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXX") + local TMP_OUTPUT=$(mktemp "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXXXX") trap "rm -rf '$TMP_OUTPUT' >/dev/null" RETURN INT TERM EXIT sigil -f "$VALIDATE_TEMPLATE" NGINX_CONF="$DOKKU_ROOT/$APP/nginx.conf" | cat -s >"$TMP_OUTPUT" diff --git a/plugins/ps/functions b/plugins/ps/functions index d3428897d..2272506c5 100755 --- a/plugins/ps/functions +++ b/plugins/ps/functions @@ -71,7 +71,7 @@ generate_scale_file() { if [[ ! -f $DOKKU_SCALE_FILE ]]; then dokku_log_info1_quiet "DOKKU_SCALE file not found in app image. Generating one based on Procfile..." - local GENERATE_SCALE_FILE_TMP_WORK_DIR=$(mktemp -d "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXX") + local GENERATE_SCALE_FILE_TMP_WORK_DIR=$(mktemp -d "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXXXX") trap "rm -rf '$GENERATE_SCALE_FILE_TMP_WORK_DIR' >/dev/null" RETURN if [[ -f $DOKKU_PROCFILE ]]; then diff --git a/plugins/scheduler-docker-local/check-deploy b/plugins/scheduler-docker-local/check-deploy index c43e19ae9..57321ff1e 100755 --- a/plugins/scheduler-docker-local/check-deploy +++ b/plugins/scheduler-docker-local/check-deploy @@ -80,7 +80,7 @@ scheduler-docker-local-check-deploy() { # use this number of retries for checks local ATTEMPTS="${DOKKU_CHECKS_ATTEMPTS:-5}" - local CHECK_DEPLOY_TMP_WORK_DIR=$(mktemp -d "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXX") + local CHECK_DEPLOY_TMP_WORK_DIR=$(mktemp -d "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXXXX") local CHECKS_FILENAME=${CHECK_DEPLOY_TMP_WORK_DIR}/CHECKS local IMAGE_TAG="$(get_running_image_tag "$APP")" local IMAGE=$(get_deploying_app_image_name "$APP" "$IMAGE_TAG") diff --git a/plugins/scheduler-docker-local/scheduler-inspect b/plugins/scheduler-docker-local/scheduler-inspect index 43263168f..5fd7e8e2e 100755 --- a/plugins/scheduler-docker-local/scheduler-inspect +++ b/plugins/scheduler-docker-local/scheduler-inspect @@ -13,7 +13,7 @@ scheduler-docker-local-scheduler-inspect() { return fi - local TMP_INSPECT_CMD=$(mktemp "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXX") + local TMP_INSPECT_CMD=$(mktemp "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXXXX") trap "rm -rf '$TMP_INSPECT_CMD' >/dev/null" RETURN INT TERM EXIT cat >"$TMP_INSPECT_CMD" </dev/null" RETURN INT TERM EXIT while read -r key; do diff --git a/plugins/ssh-keys/subcommands/add b/plugins/ssh-keys/subcommands/add index e0dcde14b..9be754d56 100755 --- a/plugins/ssh-keys/subcommands/add +++ b/plugins/ssh-keys/subcommands/add @@ -10,7 +10,7 @@ add_keys() { [[ "$1" == "$cmd" ]] && shift 1 declare NAME="$1" KEY_FILE="$2" local TMP_KEY_FILE - TMP_KEY_FILE=$(mktemp "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXX") + TMP_KEY_FILE=$(mktemp "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXXXX") trap "rm -rf '$TMP_KEY_FILE' >/dev/null" RETURN INT TERM EXIT if [[ -n "$KEY_FILE" ]]; then diff --git a/plugins/tar/functions b/plugins/tar/functions index 3a7923549..ce5e73c26 100755 --- a/plugins/tar/functions +++ b/plugins/tar/functions @@ -11,7 +11,7 @@ tar_build() { shift 1 # clean up after ourselves - local TAR_BUILD_TMP_WORK_DIR=$(mktemp -d "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXX") + local TAR_BUILD_TMP_WORK_DIR=$(mktemp -d "/tmp/dokku-${DOKKU_PID}-${FUNCNAME[0]}.XXXXXX") trap "rm -rf '$TAR_BUILD_TMP_WORK_DIR' >/dev/null" RETURN INT TERM EXIT # extract tar file diff --git a/tests/bats-exec-test-single b/tests/bats-exec-test-single index 18d525840..426f30581 100755 --- a/tests/bats-exec-test-single +++ b/tests/bats-exec-test-single @@ -43,7 +43,7 @@ main() { exit 1 fi - TMP_OUTPUT=$(mktemp "/tmp/${FUNCNAME[0]}.XXXX") + TMP_OUTPUT=$(mktemp "/tmp/${FUNCNAME[0]}.XXXXXX") trap 'rm -rf "$TMP_OUTPUT" >/dev/null' RETURN INT TERM EXIT eval "$(grep "export BATS_TEST_PATTERN"