Skip to content

Commit dac8a92

Browse files
authored
fix(middleware-flexible-checksums): append content-encoding aws-chunked instead of replacing (#4368)
1 parent 30b45af commit dac8a92

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

packages/middleware-flexible-checksums/src/flexibleChecksumsMiddleware.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ describe(flexibleChecksumsMiddleware.name, () => {
3333
const mockMiddlewareConfig = { input: mockInput, requestChecksumRequired: false };
3434

3535
const mockBody = { body: "mockRequestBody" };
36-
const mockHeaders = { "content-length": 100 };
36+
const mockHeaders = { "content-length": 100, "content-encoding": "gzip" };
3737
const mockRequest = { body: mockBody, headers: mockHeaders };
3838
const mockArgs = { request: mockRequest } as BuildHandlerArguments<any>;
3939
const mockResult = { response: { body: "mockResponsebody" } };
@@ -146,7 +146,7 @@ describe(flexibleChecksumsMiddleware.name, () => {
146146
headers: {
147147
...mockHeaders,
148148
"content-length": undefined,
149-
"content-encoding": "aws-chunked",
149+
"content-encoding": "gzip,aws-chunked",
150150
"transfer-encoding": "chunked",
151151
"x-amz-decoded-content-length": mockHeaders["content-length"],
152152
"x-amz-content-sha256": "STREAMING-UNSIGNED-PAYLOAD-TRAILER",

packages/middleware-flexible-checksums/src/flexibleChecksumsMiddleware.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,9 @@ export const flexibleChecksumsMiddleware =
5151
});
5252
updatedHeaders = {
5353
...headers,
54-
"content-encoding": "aws-chunked",
54+
"content-encoding": headers["content-encoding"]
55+
? `${headers["content-encoding"]},aws-chunked`
56+
: "aws-chunked",
5557
"transfer-encoding": "chunked",
5658
"x-amz-decoded-content-length": headers["content-length"],
5759
"x-amz-content-sha256": "STREAMING-UNSIGNED-PAYLOAD-TRAILER",

0 commit comments

Comments
 (0)