-
Notifications
You must be signed in to change notification settings - Fork 150
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
feat: Adds support for Keycloak 26.1 #1256
base: main
Are you sure you want to change the base?
feat: Adds support for Keycloak 26.1 #1256
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.
Thank you for the fix. , the issue on the ci seems to not be feasible, as we are also experiencing the same failure for those keycloak versions. I would appreciate it if you could look into the root cause.
For the lint-other-files step, ubuntu-latest jumped to ubuntu-24 which did not support python 3.7, you can pin the version to ubuntu-22.04
Hi @AssahBismarkabah, |
4636108
to
9cb1421
Compare
9cb1421
to
274b35a
Compare
Quality Gate passedIssues Measures |
@maximilian-krauss I also noticed that too, but the error below occurs locally using kc-26.1 at
|
Yes, I also can reproduce the ImportAuthenticationFlowsIT failure, but only if I run it the whole suite via maven. If I execute them via IntelliJ they work just fine |
On my side it also fails when just running the Lines 762 to 764 in 1e3badb
Why should we find two configs for the same alias? |
What this PR does / why we need it:
Adds support for Keycloak 26.1.
Which issue this PR fixes: fixes #1253
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