-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Bump Keycloak version to 24.0.4 #40662
Conversation
This comment has been minimized.
This comment has been minimized.
🙈 The PR is closed and the preview is expired. |
This comment has been minimized.
This comment has been minimized.
Sorry, I thought I tried most of the integration tests :-), will look at it tomorrow |
749aad2
to
52207f7
Compare
It was just an error message that got changed in a negative test for the unsupported token exchange feature |
Status for workflow
|
Status for workflow
|
Let me merge this one |
This is fail of coordination between Keycloak and Quarkus. Such a big change should have been merged into main much sooner, https://github.com/keycloak/keycloak/releases/tag/24.0.0 was released Mar 4. I don't like such a big change coming in without bake time. |
This upgrade is possible with thanks to @pedroigor suggesting to use a JSON file at the startup which allows Keycloak to avoid enforcing that an attribute like
email
is required during the new user registration or authentication.