-
Notifications
You must be signed in to change notification settings - Fork 92
2916 #2924
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
2916 #2924
Conversation
…lready covered by ^/api/automation/v[0-9]+/.+
…asses for `FilterAfterContributor` and `FilterBeforeContributor`.
… configurations. Refactor `SecurityConfiguration` to leverage these for improved modularity and extensibility.
…lasses in `SecurityConfiguration`.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR refactors the security configuration system to use a modular, contributor-based approach for managing web security rules across different modules. The refactoring introduces new interfaces and moves hardcoded security configurations into pluggable contributors, improving modularity and maintainability.
- Replaced hardcoded security rules with contributor-based configuration system
- Introduced new interfaces (AuthorizeHttpRequestContributor, CsrfContributor, SpaWebFilterContributor, etc.) for modular security configuration
- Renamed and moved authentication filters and providers to follow consistent naming conventions
Reviewed Changes
Copilot reviewed 42 out of 42 changed files in this pull request and generated 2 comments.
Show a summary per file
File | Description |
---|---|
platform-security-web API interfaces | Defines new contributor interfaces for modular security configuration |
SecurityConfiguration.java | Refactored to use contributor pattern instead of hardcoded rules |
Various authentication filters | Renamed from generic names to module-specific names and updated parent class references |
Contributor implementations | Added module-specific implementations of the new contributor interfaces |
Build files | Added dependencies on platform-security-web-api module |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
...r/src/main/java/com/bytechef/ai/mcp/server/security/web/config/McpServerCsrfContributor.java
Show resolved
Hide resolved
...com/bytechef/ai/mcp/server/security/web/config/McpServerAuthorizeHttpRequestContributor.java
Show resolved
Hide resolved
|
No description provided.