Skip to content

Commit a66eda9

Browse files
committed
fix(middleware-flexible-checksums): use union for new config types
1 parent a993838 commit a66eda9

File tree

2 files changed

+18
-8
lines changed

2 files changed

+18
-8
lines changed

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

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,11 @@
11
import { normalizeProvider } from "@smithy/util-middleware";
22

3-
import { DEFAULT_REQUEST_CHECKSUM_CALCULATION, DEFAULT_RESPONSE_CHECKSUM_VALIDATION } from "./constants";
3+
import {
4+
DEFAULT_REQUEST_CHECKSUM_CALCULATION,
5+
DEFAULT_RESPONSE_CHECKSUM_VALIDATION,
6+
RequestChecksumCalculation,
7+
ResponseChecksumValidation,
8+
} from "./constants";
49
import { resolveFlexibleChecksumsConfig } from "./resolveFlexibleChecksumsConfig";
510

611
jest.mock("@smithy/util-middleware");
@@ -25,8 +30,8 @@ describe(resolveFlexibleChecksumsConfig.name, () => {
2530

2631
it("normalizes client checksums configuration", () => {
2732
const mockInput = {
28-
requestChecksumCalculation: "WHEN_REQUIRED",
29-
responseChecksumValidation: "WHEN_REQUIRED",
33+
requestChecksumCalculation: RequestChecksumCalculation.WHEN_REQUIRED,
34+
responseChecksumValidation: ResponseChecksumValidation.WHEN_REQUIRED,
3035
};
3136
const resolvedConfig = resolveFlexibleChecksumsConfig(mockInput);
3237
expect(resolvedConfig).toEqual(mockInput);

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

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,28 @@
11
import { Provider } from "@smithy/types";
22
import { normalizeProvider } from "@smithy/util-middleware";
33

4-
import { DEFAULT_REQUEST_CHECKSUM_CALCULATION, DEFAULT_RESPONSE_CHECKSUM_VALIDATION } from "./constants";
4+
import {
5+
DEFAULT_REQUEST_CHECKSUM_CALCULATION,
6+
DEFAULT_RESPONSE_CHECKSUM_VALIDATION,
7+
RequestChecksumCalculation,
8+
ResponseChecksumValidation,
9+
} from "./constants";
510

611
export interface FlexibleChecksumsInputConfig {
712
/**
813
* Determines when a checksum will be calculated for request payloads.
914
*/
10-
requestChecksumCalculation?: string | Provider<string>;
15+
requestChecksumCalculation?: RequestChecksumCalculation | Provider<RequestChecksumCalculation>;
1116

1217
/**
1318
* Determines when checksum validation will be performed on response payloads.
1419
*/
15-
responseChecksumValidation?: string | Provider<string>;
20+
responseChecksumValidation?: ResponseChecksumValidation | Provider<ResponseChecksumValidation>;
1621
}
1722

1823
export interface FlexibleChecksumsResolvedConfig {
19-
requestChecksumCalculation: Provider<string>;
20-
responseChecksumValidation: Provider<string>;
24+
requestChecksumCalculation: Provider<RequestChecksumCalculation>;
25+
responseChecksumValidation: Provider<ResponseChecksumValidation>;
2126
}
2227

2328
export const resolveFlexibleChecksumsConfig = <T>(

0 commit comments

Comments
 (0)