Merge pull request #117 from CircleCI-Public/address-shellcheck-errors

fix: addressed shellcheck errors
CircleCI-Public · Aug 24, 2022 · 4313421 · 4313421
2 parents 1c840bd + 7945caa
commit 4313421
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/src/scripts/assume-role-with-web-identity.sh b/src/scripts/assume-role-with-web-identity.sh
@@ -16,13 +16,12 @@ if [ ! "$(command -v aws)" ]; then
     exit 1
 fi
 
-# shellcheck disable=SC2086,SC2034
 read -r AWS_ACCESS_KEY_ID AWS_SECRET_ACCESS_KEY AWS_SESSION_TOKEN <<EOF
 $(aws sts assume-role-with-web-identity \
---role-arn ${PARAM_AWS_CLI_ROLE_ARN} \
---role-session-name ${PARAM_ROLE_SESSION_NAME} \
---web-identity-token ${CIRCLE_OIDC_TOKEN} \
---duration-seconds ${PARAM_SESSION_DURATION} \
+--role-arn "${PARAM_AWS_CLI_ROLE_ARN}" \
+--role-session-name "${PARAM_ROLE_SESSION_NAME}" \
+--web-identity-token "${CIRCLE_OIDC_TOKEN}" \
+--duration-seconds "${PARAM_SESSION_DURATION}" \
 --query 'Credentials.[AccessKeyId,SecretAccessKey,SessionToken]' \
 --output text)
 EOF