Skip to content
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

Updates for Keycloak 24&25 #987

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

jermarchand
Copy link
Contributor

Hi,

The main feature in Keycloak 24 is "Supported user profile and progressive profiling" (https://www.keycloak.org/2024/03/keycloak-2400-released.html).
There is a "Breaking changes to the User Profile SPI".

Initialy I tried to update tests to be compatible 'pre' and 'post' version 24.
But too much use of skipIfVersionIsLessThanOrEqualTo method makes reading the tests complicated.
I don't have enougth time to do it, so I create this PR WITHOUT testing versions less than 24 AND with "Breaking changes" on User Profile.

I propose this PR with :

  • Updates for User Profile (tests & CRUD part)
  • Update the CustomUserFederationExample (Keycloak 25)
  • add saml_organization to SAML clients tests (just for version Keycloak 25)
  • add sync_mode to Idp tests (Otherwise tests failed)
  • Update CI jobs ( use Github Container Registry )

It's "Work In Progress", I have to :

  • update the documentation
  • update the examples

All comments are welcome

@jermarchand jermarchand marked this pull request as draft July 30, 2024 10:29
@mkyc
Copy link

mkyc commented Aug 30, 2024

@jermarchand do you think you're going to proceed with this PR? It is very promising and I'd like to use those features you added.

@Sanhajio
Copy link

Is this pull request still relevant ?

@aryakumargeorge
Copy link

aryakumargeorge commented Nov 8, 2024

Could this PR be progressed to support version 24? thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants