Skip to content

Conversation

@spencergibb
Copy link
Member

...lifier.

fixes spring-cloud/spring-cloud-netflix#267

@dsyer is this along the lines of what you were thinking?

@dsyer
Copy link
Contributor

dsyer commented Mar 20, 2015

I like the @LoadBalanced. Where does it get used though? Don't you need to inject it somewhere?

@spencergibb
Copy link
Member Author

We don't inject RestTemplate anywhere, just make it available to users. When you want the load balanced RestTemplate you do:

@Autowired
@LoadBalanced
private RestTemplate loadBalanced;

@dsyer
Copy link
Contributor

dsyer commented Mar 20, 2015

OK, I like. Can we do the same thing with OAuth2RestTemplate?

@spencergibb
Copy link
Member Author

yup

@spencergibb
Copy link
Member Author

in the case of OAuth2RestTemplate should we create both one with @LoadBalanced and one without?

@spencergibb
Copy link
Member Author

see spring-attic/spring-cloud-security#51 for OAuth2RestTemplate.

@spencergibb spencergibb deleted the load-balanced-rest-template branch March 20, 2015 17:35
spencergibb pushed a commit that referenced this pull request Apr 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove ambiguity with RestTemplate as to if it includes ribbon or not

3 participants