Skip to content

Document accepting an organization invitation through an identity provider - #52755

Merged
pedroigor merged 1 commit into
keycloak:mainfrom
SferaDev:docs/org-invite-broker-membership
Oct 13, 2026
Merged

pedroigor merged 1 commit into
keycloak:mainfrom
SferaDev:docs/org-invite-broker-membership

Conversation

@SferaDev

@SferaDev SferaDev commented Sep 14, 2026 •

Copy link
Copy Markdown
Contributor

Follow-up to #52183, as requested in #52183 (comment)

Mentions in the invitation docs that an invitee can also sign up through an identity provider, that the membership is unmanaged, that the account email has to match the invitation, and that the link plus that email is enough to join whichever identity provider was used.

Relates to #52181

@SferaDev
SferaDev force-pushed the docs/org-invite-broker-membership branch from 6c8bb3b to 3dc5547 Compare September 14, 2026 15:57
@SferaDev
SferaDev marked this pull request as ready for review September 14, 2026 15:59
@SferaDev
SferaDev requested a review from a team as a code owner September 14, 2026 15:59
Copilot AI balanced review requested due to automatic review settings September 14, 2026 15:59

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

The documentation must clarify that already-linked identity-provider accounts do not accept invitations through this flow.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

Documents accepting organization invitations through first-time identity-provider login.

Changes:

  • Explains unmanaged membership and email matching.
  • Warns that the broker choice is unrestricted.
  • One documented claim overlooks already-linked broker accounts.
File summaries
File Description
docs/documentation/server_admin/topics/organizations/managing-members.adoc Adds identity-provider invitation guidance.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Balanced

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Copilot AI review requested due to automatic review settings September 14, 2026 16:05
@SferaDev
SferaDev force-pushed the docs/org-invite-broker-membership branch from 3dc5547 to ffc222b Compare September 14, 2026 16:05

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

The documentation accurately reflects the implemented invitation flow and its constraints.

Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0 new
  • Review effort level: Balanced

Copilot AI review requested due to automatic review settings September 15, 2026 14:59

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.

Warning

Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 3 comments.

Comment on lines +112 to +113
IMPORTANT: Anyone with the invitation link who creates an account through any identity provider using the invited email address
joins the organization.

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

The documentation accurately reflects the implemented invitation flow and its limitations.

Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0 new
  • Review effort level: Balanced

Copilot AI review requested due to automatic review settings September 16, 2026 04:52

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

The documentation incorrectly implies every realm identity provider can consume invitations regardless of its configured first-login flow.

Get a fresh assessment by requesting another Copilot review.

Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Balanced

completes, but only if the email of the account matches the email address used to send the invitation. A user already linked to the
identity provider they choose skips that flow, so the invitation is not accepted and stays pending until it expires.

IMPORTANT: The invitation is accepted through any identity provider in the realm, not only those associated with the organization.
…vider

The invitation docs only described the registration form. Explain that the
invitee can return to the login page and authenticate through an identity
provider instead, that the resulting membership is unmanaged, and that the
email of the account has to match the address the invitation was sent to.

Note that the invitation is accepted through any identity provider in the
realm, and that a user already linked to the provider skips first broker
login, so the invitation stays pending until it expires.

Signed-off-by: Alexis Rico <sferadev@gmail.com>
Copilot AI review requested due to automatic review settings September 16, 2026 07:05
@SferaDev
SferaDev force-pushed the docs/org-invite-broker-membership branch from fb2e5f2 to 8150c20 Compare September 16, 2026 07:05

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

The documentation accurately reflects the implemented invitation flow and its limitations.

Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0 new
  • Review effort level: Balanced

@vramik vramik 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.

Thank you @SferaDev for the PR.

@pedroigor
pedroigor merged commit 3fcada6 into keycloak:main Oct 13, 2026
71 checks passed
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.

4 participants