fix: streamlining the client scope update #40808
Conversation
closes: keycloak#40805 Signed-off-by: Steve Hawkins <shawkins@redhat.com>
|
@shawkins thank you for taking the time I really appreciate it. |
For me using ~250 simple realms like your script created it cut the import time by about 35%. |
|
@shawkins I can confirm this reduces import-realm duration from 49sec to 35sec when you have ~1000 realms. |
|
Based upon the review from @yelhouti I'll go ahead and merge this - actually still need a review from @keycloak/maintainers |
|
-1 as it is a performance optimization only, and the cost of backporting and potential breakage is IMHO not worth it. If any, only backport it to 26.3. Backporting might help in catching regressions and making the person reporting it happy. |
Prevents the client scope update logic from deleting, then re-adding the same client scope.
closes: #40805
cc @yelhouti