Skip to content

Add expect100ContinueTimeoutMs to ClientConfiguration#3800

Open
kai-ion wants to merge 3 commits intomainfrom
timeoutConfig
Open

Add expect100ContinueTimeoutMs to ClientConfiguration#3800
kai-ion wants to merge 3 commits intomainfrom
timeoutConfig

Conversation

@kai-ion
Copy link
Copy Markdown
Collaborator

@kai-ion kai-ion commented Apr 17, 2026

Issue #, if available:

Description of changes:

Expose CURLOPT_EXPECT_100_TIMEOUT_MS through ClientConfiguration so customers can configure the 100-Continue timeout without needing a custom HTTP client factory. This is useful when operating behind proxies that introduce network delays, where the default 1000ms timeout can cause IncompleteBody errors on uploads.

Default of 0 preserves existing behavior (curl's built-in 1000ms).

Check all that applies:

  • Did a review by yourself.
  • Added proper tests to cover this PR. (If tests are not applicable, explain.)
  • Checked if this PR is a breaking (APIs have been changed) change.
  • Checked if this PR will not introduce cross-platform inconsistent behavior.
  • Checked if this PR would require a ReadMe/Wiki update.

Check which platforms you have built SDK on to verify the correctness of this PR.

  • Linux
  • Windows
  • Android
  • MacOS
  • IOS
  • Other Platforms

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Expose CURLOPT_EXPECT_100_TIMEOUT_MS through ClientConfiguration so
customers can configure the 100-Continue timeout without needing a
custom HTTP client factory. This is useful when operating behind
proxies that introduce network delays, where the default 1000ms
timeout can cause IncompleteBody errors on uploads.

Default of 0 preserves existing behavior (curl's built-in 1000ms).
Comment thread src/aws-cpp-sdk-core/source/http/curl/CurlHandleContainer.cpp Outdated
Comment thread src/aws-cpp-sdk-core/source/http/curl/CurlHandleContainer.cpp Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants