Skip to content

Commit d015230

Browse files
author
Thomas Wunderlich
committed
Enclose variables in the linux userdata in quotes
This prevents word splitting or globbing. See shellcheck error SC2046 for details on the error: https://github.com/koalaman/shellcheck/wiki/SC2046
1 parent 43fe73f commit d015230

File tree

1 file changed

+12
-12
lines changed

1 file changed

+12
-12
lines changed

templates/aws-stack.yml

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -973,8 +973,8 @@ Resources:
973973
Content-Type: text/x-shellscript; charset="us-ascii"
974974
#!/bin/bash -v
975975
BUILDKITE_STACK_NAME="${AWS::StackName}" \
976-
BUILDKITE_STACK_VERSION=%v \
977-
BUILDKITE_SCALE_IN_IDLE_PERIOD=${ScaleInIdlePeriod} \
976+
BUILDKITE_STACK_VERSION="%v" \
977+
BUILDKITE_SCALE_IN_IDLE_PERIOD="${ScaleInIdlePeriod}" \
978978
BUILDKITE_SECRETS_BUCKET="${LocalSecretsBucket}" \
979979
BUILDKITE_AGENT_TOKEN_PATH="${AgentTokenPath}" \
980980
BUILDKITE_AGENTS_PER_INSTANCE="${AgentsPerInstance}" \
@@ -983,19 +983,19 @@ Resources:
983983
BUILDKITE_AGENT_EXPERIMENTS="${BuildkiteAgentExperiments}" \
984984
BUILDKITE_AGENT_RELEASE="${BuildkiteAgentRelease}" \
985985
BUILDKITE_QUEUE="${BuildkiteQueue}" \
986-
BUILDKITE_AGENT_ENABLE_GIT_MIRRORS_EXPERIMENT=${EnableAgentGitMirrorsExperiment} \
986+
BUILDKITE_AGENT_ENABLE_GIT_MIRRORS_EXPERIMENT="${EnableAgentGitMirrorsExperiment}" \
987987
BUILDKITE_ELASTIC_BOOTSTRAP_SCRIPT="${BootstrapScriptUrl}" \
988988
BUILDKITE_ENABLE_INSTANCE_STORAGE="${EnableInstanceStorage}" \
989989
BUILDKITE_AUTHORIZED_USERS_URL="${AuthorizedUsersUrl}" \
990-
BUILDKITE_ECR_POLICY=${ECRAccessPolicy} \
991-
BUILDKITE_TERMINATE_INSTANCE_AFTER_JOB=${BuildkiteTerminateInstanceAfterJob} \
992-
BUILDKITE_ADDITIONAL_SUDO_PERMISSIONS=${BuildkiteAdditionalSudoPermissions} \
993-
AWS_DEFAULT_REGION=${AWS::Region} \
994-
SECRETS_PLUGIN_ENABLED=${EnableSecretsPlugin} \
995-
ECR_PLUGIN_ENABLED=${EnableECRPlugin} \
996-
DOCKER_LOGIN_PLUGIN_ENABLED=${EnableDockerLoginPlugin} \
997-
DOCKER_EXPERIMENTAL=${EnableDockerExperimental} \
998-
AWS_REGION=${AWS::Region} \
990+
BUILDKITE_ECR_POLICY="${ECRAccessPolicy}" \
991+
BUILDKITE_TERMINATE_INSTANCE_AFTER_JOB="${BuildkiteTerminateInstanceAfterJob}" \
992+
BUILDKITE_ADDITIONAL_SUDO_PERMISSIONS="${BuildkiteAdditionalSudoPermissions}" \
993+
AWS_DEFAULT_REGION="${AWS::Region}" \
994+
SECRETS_PLUGIN_ENABLED="${EnableSecretsPlugin}" \
995+
ECR_PLUGIN_ENABLED="${EnableECRPlugin}" \
996+
DOCKER_LOGIN_PLUGIN_ENABLED="${EnableDockerLoginPlugin}" \
997+
DOCKER_EXPERIMENTAL="${EnableDockerExperimental}" \
998+
AWS_REGION="${AWS::Region}" \
999999
/usr/local/bin/bk-install-elastic-stack.sh
10001000
--==BOUNDARY==--
10011001
- {

0 commit comments

Comments
 (0)