Skip to content

Conversation

@sguilhen
Copy link
Contributor

…at trigger user validation run

  • fix cuts the time taken to load 10 users in the admin console in half

Closes #34050

…at trigger user validation run

Closes keycloak#34050

Signed-off-by: Stefan Guilhen <[email protected]>
@sguilhen sguilhen requested a review from a team as a code owner October 28, 2024 17:50
@sguilhen
Copy link
Contributor Author

@pedroigor Results with the fix, using an LDAP with 2k users, searching the first 10 in admin console:

  • current main - around 14s to return the users. Each user is queried twice because of missing cache for the LDAPObject
  • with this fix - around 6s to return the same amount of users. LDAPObject is now cached, and each user is queried only once.

@sguilhen sguilhen requested a review from pedroigor October 28, 2024 17:51
@pedroigor pedroigor merged commit 4690e00 into keycloak:main Oct 28, 2024
@sguilhen sguilhen deleted the 34050-slow-ldap-search branch December 12, 2025 14:53
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.

Listing federated LDAP users is very slow with import enabled

2 participants