Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.