remove-f-webflux-and-reactive-programming #129
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
TODO needs correction and testing. Removed reactive Spring security setup in favor of servlet-based security with `ServletAuthenticationFilter`. Updated API implementations to replace `WebClient` with `RestClient`, improving handling of requests and responses. Removed unused legacy classes and configurations specific to the reactive setup.
Replaced inline form data builder with DTO classes for Fitbit and Spotify OAuth requests to improve maintainability and clarity. Added new DTOs for token requests, ensuring consistent structure and easier extensibility. Also introduced support for a new "FREESTYLE" chess platform across services and database schema.
This commit removes unused or unnecessary fields such as `fideId` and `url` from various test data models and test cases. The cleanup ensures that tests only include essential fields, improving maintainability and clarity of the codebase.
Replaced WebTestClient with MockMvc for consistency across test cases and improved maintainability. Added Testcontainers configuration for better test isolation and removed redundant or outdated configurations. Simplified and parameterized repeated test logic for improved readability and efficiency.
Introduce `FlywayClearDatabaseJunitExtension` to clean and migrate databases before each test across all services. Replace `DirtiesContext` with this extension in `TestcontainersConfiguration` and ensure proper Flyway configuration with test properties. This enhances test isolation and improves database handling consistency.
Enable `build`, `defaults`, `env`, `java`, `os`, `process`, and `ssl` under the `info` section across multiple services’ configuration files. This enhances visibility of system and build metadata in the various service configurations. Also updates the authentication manager to handle multiple authentications properly.
This update removes all dependencies and logic related to auth cookies, including their handling in interceptors, back-end configurations, and logout endpoints. Additionally, it introduces a microservice overview implementation with new components, services, and models for managing and displaying application statuses in the web interface.
Replaced specific actuator request matchers with a global rule requiring ADMIN_SERVICE authority for all requests. This simplifies configuration and ensures consistent access control across the application.
MichiBaum
added
dependencies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.