Skip to content

docs: update SSO role mapping (most privileged role wins on multi-match)#2495

Merged
Charlie-Boyer merged 1 commit into
mainfrom
auto-docs/moon-landing-pr-18282
May 26, 2026
Merged

docs: update SSO role mapping (most privileged role wins on multi-match)#2495
Charlie-Boyer merged 1 commit into
mainfrom
auto-docs/moon-landing-pr-18282

Conversation

@Charlie-Boyer
Copy link
Copy Markdown
Contributor

@Charlie-Boyer Charlie-Boyer commented May 22, 2026

Problem

The SSO doc still says the first matching mapping wins when an IdP attribute matches several role mappings.

Fix

Update the doc to reflect the new behavior introduced in huggingface-internal/moon-landing#18282: the most privileged matching role is assigned, with hierarchy read < contributor < write < admin.


Note

Low Risk
Documentation-only change that updates described SSO role-mapping precedence; no runtime behavior or security logic is modified.

Overview
Updates security-sso-user-management.md to reflect that when an IdP attribute contains multiple values matching multiple role mappings, the assigned org role is now the most privileged match (with explicit hierarchy read < contributor < write < admin) rather than the first match.

Reviewed by Cursor Bugbot for commit 7ac0a64. Bugbot is set up for automated code reviews on this repo. Configure here.

When an IdP attribute matches several role mappings (e.g. user belongs to
multiple groups), the assigned role is now the highest-privileged match
rather than the first match encountered.
@HuggingFaceDocBuilderDev
Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Copy link
Copy Markdown
Member

@Pierrci Pierrci left a comment

Choose a reason for hiding this comment

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

thanks for taking care of that!

@Charlie-Boyer Charlie-Boyer merged commit ade235c into main May 26, 2026
3 checks passed
@Charlie-Boyer Charlie-Boyer deleted the auto-docs/moon-landing-pr-18282 branch May 26, 2026 08:03
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.

3 participants