Skip to content

50877 [DOC][FGAPv2] Clarify that manage and manage-members for users allow creating users#50881

Merged
ahus1 merged 4 commits into
keycloak:mainfrom
boschglobal:50877_clarify_fgapv2_manage_users_documentation
Jul 16, 2026
Merged

50877 [DOC][FGAPv2] Clarify that manage and manage-members for users allow creating users#50881
ahus1 merged 4 commits into
keycloak:mainfrom
boschglobal:50877_clarify_fgapv2_manage_users_documentation

Conversation

@sschu

@sschu sschu commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Closes #50877

…allow creating users

Signed-off-by: Sebastian Schuster <sebastian.schuster@bosch.com>
Copilot AI review requested due to automatic review settings July 14, 2026 14:06
@sschu
sschu requested a review from a team as a code owner July 14, 2026 14:06
@sschu
sschu requested a review from vramik July 14, 2026 14:07

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.

Pull request overview

Note

Copilot couldn't run its full agentic review because no GitHub Actions runner was available. Make sure your repository has a runner available to run Copilot's review, or add a copilot-setup-steps.yml file specifying one with the runs-on attribute. See the docs for more details.

Updates Fine-Grained Admin Permissions v2 documentation to clarify that certain user and group scopes also grant the ability to create users.

Changes:

  • Clarified that the manage scope on users includes user creation.
  • Added a note explaining user-creation behavior when manage is granted via group member permissions.
  • Clarified that manage-members on groups includes creating users into the group.

Signed-off-by: Sebastian Schuster <sebastian.schuster@bosch.com>
Copilot AI review requested due to automatic review settings July 14, 2026 14:29

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.

Pull request overview

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

Signed-off-by: Sebastian Schuster <sebastian.schuster@bosch.com>
Copilot AI review requested due to automatic review settings July 15, 2026 19:29

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.

Pull request overview

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

This operation applies to any child group in the group hierarchy.
This can be prevented by explicitly denying permission for specific subgroups.
| *manage-members* | Defines if a realm administrator can manage group members.
| *manage-members* | Defines if a realm administrator can manage group members and create new users that will be members of the group.

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.

The existing documentation at lines 102–105 already covers this dual requirement. The PR's new wording at line 128 is that it sounds to me that manage-members alone lets you "create new users that will be members of the group" — which some users might find incomplete. manage-members lets you create the user, but you still need manage-membership to actually place them in the group.

What about "Together with manage-membership, also allows creating new users as members of the group."?

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

Thanks @sschu for the PR, it is nice improvement to the docs. Please see my comment inline, other than that, I thinks it's good to go.

Copilot AI review requested due to automatic review settings July 16, 2026 14:02
Signed-off-by: Sebastian Schuster <sebastian.schuster@bosch.com>
@sschu
sschu force-pushed the 50877_clarify_fgapv2_manage_users_documentation branch from d66f1af to 9aba0ec Compare July 16, 2026 14: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.

Pull request overview

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

@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 @sschu

@ahus1
ahus1 merged commit 7c5aaaa into keycloak:main Jul 16, 2026
68 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.

[DOC] FGAPv2: Clarify that manage scope on users and manage-members on group include creating users

4 participants