Merge pull request #1245 from jim-p/acme-fix-1243

Add braces around ACCOUNT_URL when forming the protected parameters. Fixes #1243
7 years ago · 71013b372d
parent 338b3ba590 6b798b01a8
commit 71013b372d
1 changed files with 1 additions and 1 deletions
--- a/acme.sh
+++ b/acme.sh
@ -1787,7 +1787,7 @@ _send_signed_request() {
      if [ "$url" = "$ACME_NEW_ACCOUNT" ] || [ "$url" = "$ACME_REVOKE_CERT" ]; then
        protected="$JWK_HEADERPLACE_PART1$nonce\", \"url\": \"${url}$JWK_HEADERPLACE_PART2, \"jwk\": $jwk"'}'
      else
-        protected="$JWK_HEADERPLACE_PART1$nonce\", \"url\": \"${url}$JWK_HEADERPLACE_PART2, \"kid\": \"$ACCOUNT_URL\""'}'
+        protected="$JWK_HEADERPLACE_PART1$nonce\", \"url\": \"${url}$JWK_HEADERPLACE_PART2, \"kid\": \"${ACCOUNT_URL}\""'}'
      fi
    else
      protected="$JWK_HEADERPLACE_PART1$nonce\", \"url\": \"${url}$JWK_HEADERPLACE_PART2, \"jwk\": $jwk"'}'