Skip to content

Commit 626c3d9

Browse files
author
kai lin
committed
change retry strategy to factory
1 parent 48c3c08 commit 626c3d9

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

src/aws-cpp-sdk-core/source/client/ClientConfiguration.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -315,9 +315,9 @@ void setConfigFromEnvOrProfile(ClientConfiguration &config)
315315
long attempts = static_cast<long>(Aws::Utils::StringUtils::ConvertToInt32(numAttemptsStr.c_str()));
316316
config.credentialProviderConfig.imdsConfig.metadataServiceNumAttempts = attempts;
317317

318-
// Create IMDS-specific retry strategy based on configured attempts
318+
// Create IMDS-specific retry strategy based on configured attempts using factory pattern
319319
if (attempts > 0) {
320-
config.credentialProviderConfig.imdsRetryStrategy = Aws::MakeShared<DefaultRetryStrategy>(CLIENT_CONFIG_TAG, attempts - 1, 1000);
320+
config.credentialProviderConfig.imdsRetryStrategy = config.configFactories.retryStrategyCreateFn();
321321
}
322322
}
323323

src/aws-cpp-sdk-core/source/internal/AWSHttpResourceClient.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,7 @@ namespace Aws
4949
{
5050
static ClientConfiguration MakeDefaultHttpResourceClientConfiguration(const char *logtag, const Aws::Client::ClientConfiguration* clientConfig = nullptr)
5151
{
52+
AWS_UNREFERENCED_PARAM(clientConfig); // Parameter reserved for future IMDS configuration customization
5253
ClientConfiguration res;
5354

5455
res.maxConnections = 2;

0 commit comments

Comments
 (0)