-
Notifications
You must be signed in to change notification settings - Fork 2.5k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # 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: