-
Notifications
You must be signed in to change notification settings - Fork 142
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
Keycloak 26.0.0 #1162
base: main
Are you sure you want to change the base?
Keycloak 26.0.0 #1162
Conversation
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.
Can you please update the change log.
I just tested this branch and it worked for me. However, I didn't test any KC26 specific configuration. |
I also think we can drop the support for KC 18.x with the upgrade to KC26. |
# Conflicts: # CHANGELOG.md
…eycloak core and libraries. Rename import folder. Bump Keycloak version in .env and Dockerfile.
Quality Gate passedIssues Measures |
Tested and this works for us as well. There was one bug we noticed though caused by this change in Keycloak 26 https://www.keycloak.org/docs/latest/release_notes/index.html#identity-providers-no-longer-available-from-the-realm-representation. The realm representation no longer contains identity provider mappers and so |
I also stumbled upon this, according to https://www.keycloak.org/docs/26.0.2/upgrading/#identity-providers-no-longer-available-from-the-realm-representation we now need to query the endpoint @SmithJosh I just gave this another try and it worked for me. I did the following:
Which IdP update scenario did not work for you?
To:
|
I updated my branch https://github.com/thomasdarimont/keycloak-config-cli/tree/update/keycloak-26.0.x with the fixed IdentityProviders lookup. |
What this PR does / why we need it:
Add support for Keycloak 26.0.0
Which issue this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close that issue when PR gets merged): fixes #1160Special notes for your reviewer:
PR Readiness Checklist:
Complete these before marking the PR as
ready to review
:CHANGELOG.md
release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR