Skip to content

Configure SimpleClientHttpRequestFactory with timeouts#908

Closed
ljcanales wants to merge 1 commit intospring-projects:mainfrom
ljcanales:configure-simpleclient-timeouts
Closed

Configure SimpleClientHttpRequestFactory with timeouts#908
ljcanales wants to merge 1 commit intospring-projects:mainfrom
ljcanales:configure-simpleclient-timeouts

Conversation

@ljcanales
Copy link
Contributor

Ensure SimpleClientHttpRequestFactory correctly applies timeout configurations from ClientOptions, maintaining consistency with Apache HttpComponents and OkHttp clients.

  • Introduced SimpleClient class to encapsulate SimpleClientHttpRequestFactory setup.
  • Applied connection and read timeouts from ClientOptions.

Closes gh-907

@ljcanales ljcanales force-pushed the configure-simpleclient-timeouts branch from edf571a to 17afc4f Compare March 3, 2025 19:14
Closes spring-projectsgh-907

Signed-off-by: Luciano Canales <64669280+ljcanales@users.noreply.github.com>
@ljcanales ljcanales force-pushed the configure-simpleclient-timeouts branch from 17afc4f to ff29474 Compare March 3, 2025 19:19
@mp911de mp911de added the type: enhancement A general enhancement label Mar 3, 2025
@mp911de mp911de self-assigned this Mar 3, 2025
@mp911de mp911de added this to the 3.2.0-M1 milestone Mar 4, 2025
mp911de pushed a commit that referenced this pull request Mar 4, 2025
Closes gh-907
Original pull request: gh-908

Signed-off-by: Luciano Canales <64669280+ljcanales@users.noreply.github.com>
mp911de added a commit that referenced this pull request Mar 4, 2025
Add since tag.

See gh-907
Original pull request: gh-908
@mp911de
Copy link
Member

mp911de commented Mar 4, 2025

Thank you for your contribution. That's merged and polished now. For 4.0, we are switching to the built-in JDK HTTP Client eliminating the need for HttpUrlConnection.

@mp911de mp911de closed this Mar 4, 2025
@ljcanales
Copy link
Contributor Author

Hey @mp911de ,

Thanks for reviewing! 🚀 Do you have an estimated release date for the 3.2.0 milestone?

Also, should these changes be committed to 2.x.x as well?

@mp911de
Copy link
Member

mp911de commented Mar 5, 2025

2.x is no longer maintained. I think, we could backport the change into 3.1.x. 3.2.0-M1 will ship in April.

@mp911de mp911de modified the milestones: 3.2.0-M1, 3.1.3 Mar 5, 2025
mp911de pushed a commit that referenced this pull request Mar 5, 2025
Closes gh-907
Original pull request: gh-908

Signed-off-by: Luciano Canales <64669280+ljcanales@users.noreply.github.com>
mp911de added a commit that referenced this pull request Mar 5, 2025
Add since tag.

See gh-907
Original pull request: gh-908
mp911de added a commit that referenced this pull request Mar 5, 2025
Add since tag.

See gh-907
Original pull request: gh-908
@ljcanales
Copy link
Contributor Author

Sounds great! Thanks! I appreciate the quick update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: enhancement A general enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SimpleClientHttpRequestFactory is not configured with timeouts in ClientHttpRequestFactoryFactory

2 participants