Skip to content
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

WebClientReactiveClientCredentialsTokenResponseClient.getTokenRespons… #6105

Merged
merged 1 commit into from
Nov 19, 2018

Conversation

raphaelDL
Copy link
Contributor

WebClientReactiveClientCredentialsTokenResponseClient.getTokenResponse expects 2xx http status code

This ensures that token response is only extracted when ClientResponse has a successful status

Fixes: gh-6089

…e expects 2xx http status code

This ensures that token response is only extracted when ClientResponse has a successful status

Fixes: spring-projectsgh-6089
@rwinch rwinch self-assigned this Nov 19, 2018
@rwinch rwinch added this to the 5.2.0.M1 milestone Nov 19, 2018
@rwinch rwinch added Reactive in: oauth2 An issue in OAuth2 modules (oauth2-core, oauth2-client, oauth2-resource-server, oauth2-jose) type: bug A general bug labels Nov 19, 2018
@rwinch rwinch merged commit e1d68e4 into spring-projects:master Nov 19, 2018
@rwinch
Copy link
Member

rwinch commented Nov 19, 2018

Thanks for the PR @raphaelDL! You are crushing it 🥇

This is now merged into master and backported to 5.1.x via #6112

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: oauth2 An issue in OAuth2 modules (oauth2-core, oauth2-client, oauth2-resource-server, oauth2-jose) type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants