Support Host header overriding URI

badslug · May 13, 2022 · cf65cc9 · cf65cc9
1 parent d06025e
commit cf65cc9
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/AWSSignature4DynamicValue.js b/AWSSignature4DynamicValue.js
@@ -247,7 +247,7 @@ var AWSSignature4DynamicValue = function() {
         if (headersArray) {
           headersArray.forEach(function(header) {
             var lower = header.name.getEvaluatedString().toLowerCase()
-            if (lower.startsWith('x-amz-')) {
+            if (lower.startsWith('x-amz-') || lower === 'host') {
               headers[lower] = header.value.getEvaluatedString();
             }
           })
@@ -275,7 +275,6 @@ var AWSSignature4DynamicValue = function() {
 
         // AWS wants the URI normalized (except for s3 which is not normalized) path URL encoded according to RFC 3986.
         // Each path segment should be URI-encoded **twice** except for s3 which only gets URI-encoded once.
-        var target = uri.pathname;
         var canonicalURI = encodeComponents(uri.pathname, service === 's3' || service === 'execute-api');
 
         // Step 1