Skip to content

Commit

Permalink
Merge pull request #19 from johnlauck/multi_indices_sig_fix
Browse files Browse the repository at this point in the history
Encode ',' in path before signing
  • Loading branch information
badslug authored Oct 24, 2017
2 parents 9fa267b + b9ce2cf commit 8239a0c
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion AWSSignature4DynamicValue.js
Original file line number Diff line number Diff line change
Expand Up @@ -232,7 +232,7 @@ var AWSSignature4DynamicValue = function() {

// Step 1
var canonical = request.method + '\n' +
uri.pathname + '\n' +
uri.pathname.replace(',', '%2C') + '\n' +
getParametersString(request, uri.search) + '\n' +
canonicalHeaders.join('\n') + '\n' +
'\n' +
Expand Down

0 comments on commit 8239a0c

Please sign in to comment.