Skip to content

Commit 6066f16

Browse files
committed
fixmiddleware-flexible-checksums): use object for requestAlgorithmMember
1 parent d42a6d9 commit 6066f16

File tree

2 files changed

+9
-5
lines changed

2 files changed

+9
-5
lines changed

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

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -108,8 +108,10 @@ describe(flexibleChecksumsMiddleware.name, () => {
108108
describe("input[requestAlgorithmMember] is not defined and", () => {
109109
const mockMwConfigWithReqAlgoMember = {
110110
...mockMiddlewareConfig,
111-
requestAlgorithmMember: mockRequestAlgorithmMember,
112-
requestAlgorithmMemberHttpHeader: mockRequestAlgorithmMemberHttpHeader,
111+
requestAlgorithmMember: {
112+
name: mockRequestAlgorithmMember,
113+
httpHeader: mockRequestAlgorithmMemberHttpHeader,
114+
},
113115
};
114116

115117
it("requestChecksumCalculation is supported", async () => {

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

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -72,15 +72,17 @@ export const flexibleChecksumsMiddleware =
7272
const { request, input } = args;
7373
const { body: requestBody, headers } = request;
7474
const { base64Encoder, streamHasher } = config;
75-
const { requestChecksumRequired, requestAlgorithmMember, requestAlgorithmMemberHttpHeader } = middlewareConfig;
75+
const { requestChecksumRequired, requestAlgorithmMember } = middlewareConfig;
7676
const requestChecksumCalculation = await config.requestChecksumCalculation();
7777

78+
const requestAlgorithmMemberName = requestAlgorithmMember?.name;
79+
const requestAlgorithmMemberHttpHeader = requestAlgorithmMember?.httpHeader;
7880
// The value for input member to configure flexible checksum is not set.
79-
if (requestAlgorithmMember && !input[requestAlgorithmMember]) {
81+
if (requestAlgorithmMemberName && !input[requestAlgorithmMemberName]) {
8082
// Set requestAlgorithmMember as default checksum algorithm only if request checksum calculation is supported
8183
// or request checksum is required.
8284
if (requestChecksumCalculation === RequestChecksumCalculation.WHEN_SUPPORTED || requestChecksumRequired) {
83-
input[requestAlgorithmMember] = DEFAULT_CHECKSUM_ALGORITHM;
85+
input[requestAlgorithmMemberName] = DEFAULT_CHECKSUM_ALGORITHM;
8486
if (requestAlgorithmMemberHttpHeader) {
8587
headers[requestAlgorithmMemberHttpHeader] = DEFAULT_CHECKSUM_ALGORITHM;
8688
}

0 commit comments

Comments
 (0)