include x-amz-website-redirect-location as canonical header

jamhall · Jun 7, 2020 · 31dbfc3 · 31dbfc3
1 parent 9dcfc37
commit 31dbfc3
Showing 1 changed file with 2 additions and 9 deletions.
diff --git a/lib/middleware/authentication.js b/lib/middleware/authentication.js
@@ -57,10 +57,7 @@ module.exports = () =>
 
     const amzQueryHeaders = pickBy(
       mapKeys(ctx.query, (value, key) => key.toLowerCase()),
-      (value, key) =>
-        key.startsWith('x-amz-') &&
-        // x-amz-website-redirect-location isn't a canonical header
-        key !== 'x-amz-website-redirect-location',
+      (value, key) => key.startsWith('x-amz-'),
     );
     // x-amz-* values specified in query params take precedence over those in the headers
     Object.assign(ctx.headers, amzQueryHeaders);
@@ -125,11 +122,7 @@ module.exports = () =>
       .replace(/=(&|$)/g, ''); // remove trailing = for empty params
 
     const canonicalizedAmzHeaders = Object.keys(ctx.headers)
-      .filter(
-        headerName =>
-          headerName.startsWith('x-amz-') &&
-          headerName !== 'x-amz-website-redirect-location',
-      )
+      .filter(headerName => headerName.startsWith('x-amz-'))
       .sort()
       .map(
         headerName =>