-
Notifications
You must be signed in to change notification settings - Fork 6k
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
Support fullyAuthenticated in Kotlin authorizeHttpRequests #16162
Labels
in: config
An issue in spring-security-config
status: duplicate
A duplicate of another issue
type: enhancement
A general enhancement
Comments
Hi, @sgrimm, thanks for the report. I think this would be a reasonable addition for the 6.5 release. Are you able to submit a PR to add |
franticticktick
added a commit
to franticticktick/spring-security
that referenced
this issue
Nov 29, 2024
Thanks, @franticticktick for the PR! Closing this in favor of #16190. |
franticticktick
added a commit
to franticticktick/spring-security
that referenced
this issue
Dec 5, 2024
franticticktick
added a commit
to franticticktick/spring-security
that referenced
this issue
Dec 5, 2024
franticticktick
added a commit
to franticticktick/spring-security
that referenced
this issue
Dec 5, 2024
jzheaux
pushed a commit
that referenced
this issue
Jan 14, 2025
kwondh5217
pushed a commit
to kwondh5217/spring-security
that referenced
this issue
Feb 4, 2025
Closes spring-projectsgh-16162 Signed-off-by: Daeho Kwon <[email protected]>
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
status: duplicate
A duplicate of another issue
type: enhancement
A general enhancement
Describe the bug
Upgrading from Spring Boot 3.3.5 to 3.4.0 includes an upgrade to Spring Security 6.4, which deprecates the
authorizeRequests
block in the HTTP configuration DSL. The deprecation message suggests usingauthorizeHttpRequests
instead. ButauthorizeHttpRequests
is missing thefullyAuthenticated
property.w: file:///home/runner/work/terraware-server/terraware-server/src/main/kotlin/com/terraformation/backend/auth/SecurityConfig.kt:67:7 '@Deprecated(...) fun authorizeRequests(authorizeRequestsConfiguration: AuthorizeRequestsDsl.() -> Unit): Unit' is deprecated. Since 6.4. Use authorizeHttpRequests instead.
To Reproduce
In a Spring Boot 3.3.5 app, use a security configuration like
Upgrade to Spring Boot 3.4.0 and follow the suggestion to replace
authorizeRequests
withauthorizeHttpRequests
:Compilation will fail because
fullyAuthenticated
is undefined.Expected behavior
The suggested replacement in the deprecation message should include all the functionality of the older version or there should be a migration guide describing what to use instead.
Sample
https://github.com/sgrimm/spring-security-fullyauthenticated
SecurityConfig.kt in that repo
Workaround
Define
fullyAuthenticated
in the application code:The text was updated successfully, but these errors were encountered: