8000 Improve realm search by name (#40793) by thomasdarimont · Pull Request #40795 · keycloak/keycloak · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Improve realm search by name (#40793) #40795

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Improve realm search by name (#40793) #40795

wants to merge 1 commit into from
+22 −2

Conversation

thomasdarimont
Copy link
Contributor

Default realm search now also considers realm displayName by name. To explicitly search by name or displayName we now support the prefixes "name:" and "displayName:".

We now also consider the realm display name if set in realm searches by name.

Fixes #40793

@thomasdarimont
Improve realm search by name (keycloak#40793)
7556669 
Default realm search now also considers realm displayName by name.
To explicitly search by name or displayName we now support the prefixes "name:" and "displayName:".

We now also consider the realm display name if set in realm searches by name.

Fixes keycloak#40793

Signed-off-by: Thomas Darimont <thomas.darimont@googlemail.com>
@thomasdarimont thomasdarimont requested review from a team as code owners June 30, 2025 11:51
@edewit
Copy link
Contributor
edewit commented Jun 30, 2025

Display name can be translated, so it doesn't take that into account

@thomasdarimont
Copy link
Contributor Author
thomasdarimont commented Jun 30, 2025 •
edited
Loading

@edewit good point, yes indeed if realm admins choose to translate the realm display name, then the search will not work for those scenarios.
However, the majority of installations I have seen use a fixed realm display name in local deployments, especially in SaaS deployments with a realm per tenant model, where the realm name is often generated.
Think of a long list of realms with uuids as names but redable display names. Admins who manage the platform now needs to maintain a separate searchable list that map (technical) realm-name to realm display names. since the display name is the only way to have a human friendly "name" in that case.

ssilvert
ssilvert requested changes Jul 7, 2025
View reviewed changes
Copy link
Contributor
@ssilvert ssilvert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@thomasdarimont I think this is a nice improvement. But the user needs to know how to do the different queries from the admin UI.

We have a few places in the UI where you can search based on different criteria. See the user search for an example.

@ssilvert ssilvert self-assigned this Jul 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ssilvert ssilvert ssilvert requested changes

Requested changes must be addressed to merge this pull request.

Assignees

@ssilvert ssilvert

Labels
team/ui
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Realm search in Manage-Realms view should also include display-name
3 participants
@thomasdarimont @edewit @ssilvert
0