Skip to content

AuthenticatedReactiveAuthorizationManager support for AnonymousAuthenticationToken #6235

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
Closed
rwinch opened this issue Dec 5, 2018 · 3 comments
Closed

AuthenticatedReactiveAuthorizationManager support for AnonymousAuthenticationToken #6235

rwinch opened this issue Dec 5, 2018 · 3 comments
Labels
type: enhancement A general enhancement
Milestone
5.2.0.M1

Comments

@rwinch
Copy link
Member

rwinch commented Dec 5, 2018

Summary

AuthenticatedReactiveAuthorizationManager should filter out anonymous authentication by leveraging AuthenticationTrustResolver.
@rwinch rwinch added type: enhancement A general enhancement status: ideal-for-contribution An issue that we actively are looking for someone to help us with Reactive labels Dec 5, 2018
@rwinch rwinch added this to the 5.2.0.M1 milestone Dec 5, 2018
@rwinch rwinch mentioned this issue Dec 5, 2018
Merged
@mibo
Copy link
Contributor

mibo commented Dec 5, 2018 •
edited
Loading

Hi @rwinch I would like to take this one.

@rwinch
Copy link
Member Author

rwinch commented Dec 5, 2018

@mibo Thanks for you volunteering! The issue is yours :) Let me know if you need any help.

@mibo mibo mentioned this issue Dec 10, 2018
Merged
@mibo
Copy link
Contributor

mibo commented Dec 10, 2018

@rwinch I checked #6198 and created following PR: #6267

Can you please check if this solution fit and give feedback. Thanks.

mibo added a commit to mibo/spring-security that referenced this issue Dec 12, 2018
@mibo
Add Anonymous Support to AuthenticatedReactiveAuthorizationManager
f77cfbc 
Fixes: spring-projectsgh-6235
rwinch pushed a commit that referenced this issue Dec 12, 2018
@mibo @rwinch
Add Anonymous Support to AuthenticatedReactiveAuthorizationManager
60e3bf4 
Fixes: gh-6235
@rwinch rwinch removed status: ideal-for-contribution An issue that we actively are looking for someone to help us with labels Jan 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: enhancement A general enhancement
Projects
None yet
Milestone
5.2.0.M1
Development

No branches or pull requests
2 participants
@rwinch @mibo