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

Add OAuth2LoginSpec.authenticationFailureHandler #7051

Closed
dsyer opened this issue Jun 28, 2019 · 2 comments · Fixed by #7071

Comments

@dsyer
Copy link
Member

commented Jun 28, 2019

It appears to be possible in MVC but not Webflux.

@rwinch rwinch changed the title Provide DSL for customizing error responses in OAuth2 login with Webflux Add OAuth2LoginSpec.authenticationFailureHandler Jun 28, 2019

@rwinch

This comment has been minimized.

Copy link
Member

commented Jun 28, 2019

Similar to OAuth2LoginSpec.authenticationSuccessHandler we should add OAuth2LoginSpec.authenticationFailureHandler

@rwinch rwinch added the in: config label Jun 28, 2019

@eddumelendez

This comment has been minimized.

Copy link
Contributor

commented Jun 28, 2019

I will take a look into it :)

@rwinch rwinch self-assigned this Jul 3, 2019

eddumelendez added a commit to eddumelendez/spring-security that referenced this issue Jul 4, 2019
eddumelendez added a commit to eddumelendez/spring-security that referenced this issue Jul 21, 2019
eddumelendez added a commit to eddumelendez/spring-security that referenced this issue Jul 21, 2019
eddumelendez added a commit to eddumelendez/spring-security that referenced this issue Jul 21, 2019

@rwinch rwinch removed their assignment Jul 29, 2019

@rwinch rwinch closed this in #7071 Aug 5, 2019

rwinch added a commit that referenced this issue Aug 5, 2019
Add authenticationFailureHandler method in OAuth2LoginSpec
Allow to customize the failure handler.

Fixes gh-7051

@rwinch rwinch self-assigned this Aug 5, 2019

@rwinch rwinch added this to the 5.2.0.RC1 milestone Aug 5, 2019

kostya05983 added a commit to kostya05983/spring-security that referenced this issue Aug 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.