diff --git a/packages/middleware-retry/src/configurations.spec.ts b/packages/middleware-retry/src/configurations.spec.ts index 777c46fa3fbc..cb79a0491375 100644 --- a/packages/middleware-retry/src/configurations.spec.ts +++ b/packages/middleware-retry/src/configurations.spec.ts @@ -1,10 +1,11 @@ import { CONFIG_MAX_ATTEMPTS, + DEFAULT_MAX_ATTEMPTS, ENV_MAX_ATTEMPTS, NODE_MAX_ATTEMPT_CONFIG_OPTIONS, resolveRetryConfig, } from "./configurations"; -import { DEFAULT_MAX_ATTEMPTS, StandardRetryStrategy } from "./defaultStrategy"; +import { StandardRetryStrategy } from "./defaultStrategy"; jest.mock("./defaultStrategy"); diff --git a/packages/middleware-retry/src/configurations.ts b/packages/middleware-retry/src/configurations.ts index fcdc869a1031..ff9b97cbf8e4 100644 --- a/packages/middleware-retry/src/configurations.ts +++ b/packages/middleware-retry/src/configurations.ts @@ -1,11 +1,22 @@ import { LoadedConfigSelectors } from "@aws-sdk/node-config-provider"; import { Provider, RetryStrategy } from "@aws-sdk/types"; -import { DEFAULT_MAX_ATTEMPTS, DEFAULT_RETRY_MODE, StandardRetryStrategy } from "./defaultStrategy"; +import { StandardRetryStrategy } from "./defaultStrategy"; export const ENV_MAX_ATTEMPTS = "AWS_MAX_ATTEMPTS"; export const CONFIG_MAX_ATTEMPTS = "max_attempts"; +/** + * The default value for how many HTTP requests an SDK should make for a + * single SDK operation invocation before giving up + */ +export const DEFAULT_MAX_ATTEMPTS = 3; + +/** + * The default retry algorithm to use. + */ +export const DEFAULT_RETRY_MODE = "standard"; + export const NODE_MAX_ATTEMPT_CONFIG_OPTIONS: LoadedConfigSelectors = { environmentVariableSelector: (env) => { const value = env[ENV_MAX_ATTEMPTS]; diff --git a/packages/middleware-retry/src/defaultStrategy.spec.ts b/packages/middleware-retry/src/defaultStrategy.spec.ts index fc5d868679cd..a18c579e3b6f 100644 --- a/packages/middleware-retry/src/defaultStrategy.spec.ts +++ b/packages/middleware-retry/src/defaultStrategy.spec.ts @@ -2,9 +2,10 @@ import { HttpRequest } from "@aws-sdk/protocol-http"; import { isThrottlingError } from "@aws-sdk/service-error-classification"; import { v4 } from "uuid"; +import { DEFAULT_MAX_ATTEMPTS } from "./configurations"; import { DEFAULT_RETRY_DELAY_BASE, INITIAL_RETRY_TOKENS, THROTTLING_RETRY_DELAY_BASE } from "./constants"; import { getDefaultRetryQuota } from "./defaultRetryQuota"; -import { DEFAULT_MAX_ATTEMPTS, StandardRetryStrategy } from "./defaultStrategy"; +import { StandardRetryStrategy } from "./defaultStrategy"; import { defaultDelayDecider } from "./delayDecider"; import { defaultRetryDecider } from "./retryDecider"; import { RetryQuota } from "./types"; diff --git a/packages/middleware-retry/src/defaultStrategy.ts b/packages/middleware-retry/src/defaultStrategy.ts index f9b334504da0..ab621b0687d7 100644 --- a/packages/middleware-retry/src/defaultStrategy.ts +++ b/packages/middleware-retry/src/defaultStrategy.ts @@ -4,6 +4,7 @@ import { SdkError } from "@aws-sdk/smithy-client"; import { FinalizeHandler, FinalizeHandlerArguments, MetadataBearer, Provider, RetryStrategy } from "@aws-sdk/types"; import { v4 } from "uuid"; +import { DEFAULT_MAX_ATTEMPTS, DEFAULT_RETRY_MODE } from "./configurations"; import { DEFAULT_RETRY_DELAY_BASE, INITIAL_RETRY_TOKENS, @@ -16,17 +17,6 @@ import { defaultDelayDecider } from "./delayDecider"; import { defaultRetryDecider } from "./retryDecider"; import { DelayDecider, RetryDecider, RetryQuota } from "./types"; -/** - * The default value for how many HTTP requests an SDK should make for a - * single SDK operation invocation before giving up - */ -export const DEFAULT_MAX_ATTEMPTS = 3; - -/** - * The default retry algorithm to use. - */ -export const DEFAULT_RETRY_MODE = "standard"; - /** * Strategy options to be passed to StandardRetryStrategy */