Support Spring RestClient as TransportClientFactory

spring-cloud / spring-cloud-netflix

Integration with Netflix OSS components

http://cloud.spring.io/spring-cloud-netflix/

Apache License 2.0

4.85k stars 2.43k forks source link

Open heowc opened 2 months ago

heowc commented 2 months ago

Add related classes

Add property

Change auto-configure

RestTemplate-related settings for webclient.enabled=false have been removed. restclient.enabled makes it meaningless

Move commonly used classes

Edit the document

Added similar tests referring to RestTemplate and WebClient

Close #4257

heowc commented 2 months ago

Could you please inform me if there are any plans to phase out RestTemplate related implementations in the future? 🤔

injae-kim commented 2 months ago

heowc commented 4 weeks ago

gentle ping 😄

OlgaMaciaszek commented 3 weeks ago

Thanks @heowc, will take a look. We do not have a plan for phasing out RestTemplate at this point.

OlgaMaciaszek commented 2 weeks ago

Fixes gh-4257.

OlgaMaciaszek commented 1 week ago

Hi @heowc, I have pushed a fix in the same area: https://github.com/spring-cloud/spring-cloud-netflix/commit/b23c5873192aebaf147f0566ca4817e4ed475d24. Please merge the code and resolve any conflict before proceeding with working on this PR.