-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#3084: use factories for client utility classes initialization, better shutdown #3087
Conversation
c5a2ec7
to
f0e8a6e
Compare
728baee
to
64f90ff
Compare
m_signerProvider(Aws::MakeUnique<Aws::Auth::DefaultAuthSignerProvider>(AWS_CLIENT_LOG_TAG, signer)), | ||
m_httpClient(CreateHttpClient(configuration)), | ||
m_httpClient(CreateHttpClient( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is wild, I know...
64f90ff
to
bb864b0
Compare
*/ | ||
std::shared_ptr<RetryStrategy> retryStrategy; | ||
std::shared_ptr<RetryStrategy> retryStrategy = nullptr; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should we make a ClientConfiguration
constructor that takes a ProviderFactories
instead of push the logic to the client?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can do it later.
Additionally, I see no much extra harm in default-initing to default factories and then allowing users to over-write.
Issue #, if available:
#3084
Description of changes:
Use factories in the client config instead of pointers to the actual objects
Minor changes
Check all that applies:
Check which platforms you have built SDK on to verify the correctness of this PR.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.