Skip to content

Reactive Opaque Token Support #6519

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Feb 15, 2019
Merged

Reactive Opaque Token Support #6519

merged 1 commit into from
Feb 15, 2019

Conversation

jzheaux
Copy link
Contributor

@jzheaux jzheaux commented Feb 8, 2019

Fixes: gh-6513

jzheaux
jzheaux commented Feb 8, 2019
View reviewed changes
config/src/main/java/org/springframework/security/config/web/server/ServerHttpSecurity.java
@@ -1128,17 +1214,10 @@ public OAuth2ResourceServerSpec and() {
}

protected void configure(ServerHttpSecurity http) {
this.bearerTokenServerWebExchangeMatcher.setBearerTokenConverter(bearerTokenConverter);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that this code was extracted from JwtSpec so that it could be executed for OpaqueTokenSpec as well.

@jzheaux jzheaux requested review from jgrandja and rwinch and removed request for jgrandja February 11, 2019 22:23
@jzheaux jzheaux added status: duplicate A duplicate of another issue New Feature in: oauth2 An issue in OAuth2 modules (oauth2-core, oauth2-client, oauth2-resource-server, oauth2-jose) labels Feb 11, 2019
@jzheaux jzheaux added this to the 5.2.0.M2 milestone Feb 11, 2019
@rwinch rwinch self-assigned this Feb 15, 2019
@rwinch rwinch merged commit fba2561 into spring-projects:master Feb 15, 2019
@rwinch
Copy link
Member

rwinch commented Feb 15, 2019

Thanks for the PR @jzheaux! I have merged this into master

@jzheaux jzheaux deleted the gh-6513 branch February 15, 2019 23:20
@rwinch rwinch added the type: enhancement A general enhancement label May 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rwinch rwinch Awaiting requested review from rwinch

Assignees

@rwinch rwinch

Labels
in: oauth2 An issue in OAuth2 modules (oauth2-core, oauth2-client, oauth2-resource-server, oauth2-jose) status: duplicate A duplicate of another issue type: enhancement A general enhancement
Projects
None yet
Milestone
5.2.0.M2
Development

Successfully merging this pull request may close these issues.

Add Reactive Opaque Token Support to Resource Server
2 participants
@jzheaux @rwinch