Conversation
This is a workaround for quarkusio/quarkus#56942 Closes keycloak#53106 Signed-off-by: Václav Muzikář <vmuzikar@ibm.com>
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The workaround is narrowly scoped, preserves existing filters, and existing startup tests reject unexpected warnings.
Review effort: Balanced
Findings: None
What changed in this PR
Suppresses a false-positive Quarkus startup warning when indexing REST endpoints returning void.
Changes:
- Installs a targeted
IndexWrapperlog filter during RESTEasy configuration. - Preserves existing filtering and suppresses only the known warning signature.
| File | Description |
|---|---|
quarkus/deployment/src/main/java/org/keycloak/quarkus/deployment/KeycloakProcessor.java |
Adds the temporary Quarkus warning suppression. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
An issue to remove the workaround: #53120 |
michalvavrik
left a comment
There was a problem hiding this comment.
We may not need this workaround since original issue is only present in unreleased Keycloak (Quarkus 3.39) and based on comments in quarkusio/quarkus#56944 I hear is that the Quarkus platform version (not yet released) we shall consume may already be based on that. Let's wait.
This is a workaround for quarkusio/quarkus#56942
Closes #53106