-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Support Spring RestClient as TransportClientFactory and make it the default implementation #4257
Comments
Makes sense. |
FYI, PR: #4281 |
Thanks, @injae-kim. @heowc is working on this now. |
Update: there's still some work related to adjusting config in TransportClientFactories, that I'm working on here that has to be modified in order for RestClient to be used as the default. |
Thanks for looking into it. I think it can be modified with |
We need to integrate with Netflix's native way of handling connections through
|
Describe the solution you'd like
Complements: #4255
Consider implementing RestClientTransportClientFactory to complement existing RestTemplateClientFactory (not exactly based on RestTemplate), which is kind of deprecated.
https://docs.spring.io/spring-framework/reference/integration/rest-clients.html#rest-restclient
See note: https://docs.spring.io/spring-framework/reference/integration/rest-clients.html#rest-resttemplate
The text was updated successfully, but these errors were encountered: