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.

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

Update HandlerMappingIntrospector Usage in CORS support #16501

Open
Tracked by #16417
jzheaux opened this issue Jan 29, 2025 · 2 comments
Open
Tracked by #16417

Update HandlerMappingIntrospector Usage in CORS support #16501

jzheaux opened this issue Jan 29, 2025 · 2 comments
Assignees
@jzheaux
Labels
in: config An issue in spring-security-config type: enhancement A general enhancement
Milestone
6.5.x

Comments

@jzheaux
Copy link
Contributor

jzheaux commented Jan 29, 2025 •
edited
Loading

The CorsConfigurationConfigurer should look up a PreFlightRequestHandler instead of a HandlerMappingIntrospector. It will still be registered under the same bean name in Framework 7, so that should remain as-is.

If a PreFlightRequestHandler is found by the given name, then safely cast it to CorsConfigurationSource and construct the CorsFilter as normal.
@jzheaux jzheaux mentioned this issue Jan 29, 2025
Open
8 tasks
@jzheaux jzheaux added in: config An issue in spring-security-config type: enhancement A general enhancement labels Feb 3, 2025
@jzheaux jzheaux self-assigned this Feb 4, 2025
@jzheaux
Copy link
Contributor Author

jzheaux commented Feb 4, 2025

@evgeniycheban, this ticket is yours. Once you comment, I'll be able to assign it to you officially.

@jzheaux jzheaux added this to the 6.5.x milestone Feb 4, 2025
@evgeniycheban
Copy link
Contributor

Hi, @jzheaux, please assign it to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jzheaux jzheaux

Labels
in: config An issue in spring-security-config type: enhancement A general enhancement
Projects
Spring Security Team
Status: No status
Milestone
6.5.x
Development

No branches or pull requests
2 participants
@jzheaux @evgeniycheban