Repository navigation
Conversation
…ders The tab looked up the user's linked providers in the first page of identity providers, which the server caps at 100. When the provider was not on that page, providerId stayed undefined and building the link threw 'Missing ":providerId" param'. Fetch each linked provider by alias instead, and show the alias without a link if the provider can't be found. Closes keycloak#52482 Signed-off-by: Luca Marsella <34103646+marsluca@users.noreply.github.com>
edewit
approved these changes
Oct 29, 2026
Contributor
|
@pedroigor do you want to do it like this or do we want to have an endpoint for this? |
Contributor
|
I agree with @edewit 's first comment - fixes the issue but it is not the most efficient way. Like I said in the linked issue, I do think our backend lacks proper support for retrieving the linked identities. The same functionality exists in the account console, so I think the proper support should be added to the backend. |
Contributor
|
Closing in favor of #54617 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #52482
The Identity provider links tab looked up the user's linked providers in
GET /identity-provider/instances, which returns at most 100 providers when nomaxis given. If a user was linked to a provider outside that first page,providerIdstayed undefined and building the provider link threwMissing ":providerId" param, so the tab only showed that error.The tab now fetches each linked provider by alias, the same way
UserSelectandGroupSelectresolve their items. A user usually has only a few links, so this is a few small requests. If a provider can't be found, the alias is shown without a link instead of breaking the tab.I kept the fix in the admin console. The server-side endpoint suggested in the issue would remove this lookup as well, but it needs a new admin API.
Testing:
test/users/main.spec.ts: a realm with 101 providers and a user linked to the last one. It fails on main with the error above and passes with this change.AI-assisted (Claude Code).