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 loginPage() to DSL in reactive oauth2Login() #15674

Closed
Tracked by #15299
sjohnr opened this issue Aug 22, 2024 · 0 comments
Closed
Tracked by #15299

Add loginPage() to DSL in reactive oauth2Login() #15674

sjohnr opened this issue Aug 22, 2024 · 0 comments
Assignees
Labels
in: config An issue in spring-security-config type: enhancement A general enhancement
Milestone

Comments

@sjohnr
Copy link
Member

sjohnr commented Aug 22, 2024

We should add the ability to specify the loginPage() for the oauth2Login() DSL in ServerHttpSecurity while preserving other defaults. In particular, we should ensure that content negotiation of the ServerAuthenticationEntryPoint for non text/html media types is preserved similar to loginPage() of formLogin().

Note: This will improve alignment of configuration with the non-reactive stack.

@sjohnr sjohnr self-assigned this Aug 22, 2024
@sjohnr sjohnr converted this from a draft issue Aug 22, 2024
@sjohnr sjohnr moved this to Prioritized in Spring Security Team Aug 22, 2024
@sjohnr sjohnr added this to the 6.4.x milestone Aug 22, 2024
@sjohnr sjohnr moved this from Prioritized to In Progress in Spring Security Team Sep 11, 2024
@sjohnr sjohnr closed this as completed in 51c226f Sep 11, 2024
@sjohnr sjohnr modified the milestones: 6.4.x, 6.4.0-M4 Sep 11, 2024
@sjohnr sjohnr moved this from In Progress to Done in Spring Security Team Sep 12, 2024
@sjohnr sjohnr added in: config An issue in spring-security-config type: enhancement A general enhancement labels Oct 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: config An issue in spring-security-config type: enhancement A general enhancement
Projects
Status: Done
Development

No branches or pull requests

1 participant