8000 Fix crash and inconsistent matching in account console linked-account… by sguilhen · Pull Request #54652 · keycloak/keycloak · GitHub
[go: up one dir, main page]

Skip to content

Fix crash and inconsistent matching in account console linked-account… - #54652

Merged
pedroigor merged 1 commit into
keycloak:mainfrom
sguilhen:54647-linked-accounts-search-fix
Nov 6, 2026
Merged

pedroigor merged 1 commit into
keycloak:mainfrom
sguilhen:54647-linked-accounts-search-fix

Conversation

@sguilhen
@sguilhen sguilhen commented Nov 4, 2026
Copy link
Copy Markdown
Contributor

…s search

Closes #54647

@sguilhen
sguilhen requested a review from a team as a code owner November 4, 2026 19:05
Copilot AI balanced review requested due to automatic review settings November 4, 2026 19:05
@sguilhen
sguilhen requested a review from pedroigor November 4, 2026 19:06
Copilot AI left a comment
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

Locale-dependent lowercasing causes GIT* to miss github and gitlab under a Turkish JVM locale; the method should use Locale.ROOT consistently.

1 open issue
What changed in this PR

This PR addresses #54647 by preventing linked-account search crashes on single-character inputs and aligning case-insensitive matching across linked and unlinked providers.

Changes:

  • Guard paired-delimiter parsing and handle a lone quote before querying unlinked providers.
  • Make non-exact linked-provider searches case-insensitive.
  • Add regression tests for uppercase searches and bare * and " inputs.
File Description
tests/​base/​src/​test/​java/​org/​keycloak/​tests/​account/​LinkedAccountsRestServiceTest.java Adds case-insensitive and edge-case search coverage for both provider lists.
services/​src/​main/​java/​org/​keycloak/​services/​resources/​account/​LinkedAccountsResource.java Guards delimiter parsing, normalizes lone-quote searches, and adds case-insensitive matching.

Review effort: Balanced

@sguilhen
sguilhen force-pushed the 54647-linked-accounts-search-fix branch from d4406d0 to 22800b2 Compare November 5, 2026 12:33
Copilot AI balanced review requested due to automatic review settings November 5, 2026 12:33
Copilot AI left a comment
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

No active review issues were identified in the linked-account search changes.

No open issues 🎉

1 resolved since last review

Review effort: Balanced

…s search

Closes keycloak#54647

Signed-off-by: Stefan Guilhen <sguilhen@redhat.com>
@pedroigor
pedroigor force-pushed the 54647-linked-accounts-search-fix branch from 22800b2 to 1d23fae Compare November 6, 2026 13:11
Copilot AI balanced review requested due to automatic review settings November 6, 2026 13:11
@pedroigor
pedroigor enabled auto-merge (rebase) November 6, 2026 13:11
Copilot AI left a comment
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

No active review issues remain for the linked-account search fix and case-insensitive matching updates.

No open issues 🎉

Review effort: Balanced

@pedroigor
pedroigor merged commit 77a8f12 into keycloak:main Nov 6, 2026
96 of 165 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Account console linked-accounts endpoint crashes on a bare * or " search value

3 participants

0