Merge pull request #334 from ahamilton55/ah/fix-multiple-repos

fix: fix jq command when working with an existing Docker config.json file
CircleCI-Public · Mar 6, 2024 · 08770f8 · 08770f8
2 parents 00f044a + 26dea5f
commit 08770f8
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/scripts/ecr_login.sh b/src/scripts/ecr_login.sh
@@ -38,9 +38,10 @@ configure_config_json(){
     if [ ! -s "${CONFIG_FILE}" ]; then
         jq_flag="-n"
     fi
-        jq "${jq_flag}" --arg url "${AWS_ECR_VAL_ACCOUNT_URL}" \
-        --arg helper "ecr-login" '.credHelpers[$url] = $helper' \
-        "${CONFIG_FILE}" > temp.json && mv temp.json "${CONFIG_FILE}"
+
+    jq ${jq_flag} --arg url "${AWS_ECR_VAL_ACCOUNT_URL}" \
+      --arg helper "ecr-login" '.credHelpers[$url] = $helper' \
+      "${CONFIG_FILE}" > temp.json && mv temp.json "${CONFIG_FILE}"
 }
 
 install_aws_ecr_credential_helper(){