Repository navigation
docs(permissions): the SSO page describes how IdP groups grant permission sets - #31664
Merged
Merged
Conversation
…sion sets The SSO page gains a section on the SSO half of IdP group mapping: where the groups claim is configured (the provider's own mapping.extraFields.groups), how a platform administrator saves and deletes a mapping through the shared set-group-mapping / delete-group-mapping door with providerId, which sets can be granted, how grants follow the IdP at every sign-in, and what a failure does. The SCIM credentials section links to it: one mapping model, two sources. Claude-Session: https://claude.ai/code/session_01ADHkmSWdZY3vu8awytLsrR Co-authored-by: Claude <noreply@anthropic.com>
Contributor
Author
|
domain:server#2 · os-steve · session_01ADHkmQKZzp4DLtqiCkZZRM 维护者速读
席位核对 — ACCEPT · domain:server#2 · Head-sha
|
9 tasks done
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Requested by Steve · project thread
Fixes #31658
Clause-②: no
Metadata-protocol: none
维护者速读
content/docs/permissions/sso.mdx(新节,逐条对照 feat(plugin-auth): an SSO sign-in grants the permission set its IdP group is mapped to #31657 的代码与 changeset)、content/docs/permissions/authentication.mdx(一行回链)。pnpm check:doc-anchors,预期两处新锚点链接都能解析、全绿。What changes
content/docs/permissions/sso.mdxgains one section, Grant permission sets from IdP groups, describing the SSO half of IdP group mapping that PR #31657 landed. Every statement was read from the merged code (sso-group-projection.ts,idp-group-mapping-admin.ts, the route-ledger notes) and #31657's changeset, not written from memory:mapping.extraFields.groups(onoidcConfig.mapping/samlConfig.mappingatPOST /api/v1/auth/sso/register); a string or a list; a provider whose mapping fills nogroupssigns in as before;POST /api/v1/auth/admin/scim/set-group-mappingwithproviderId(exactly one ofconnectionId/providerId; exactly one ofgroupExternalId/groupDisplayName), removed throughdelete-group-mapping;400 VALIDATION_FAILEDonproviderId/permissionSet,503with SSO off or no security service);errorand never fails the sign-in.The section links the SCIM credentials section of
content/docs/permissions/authentication.mdx, and that section gains one back-link line (one mapping model, two sources). Rules only; no bypass narrative.Tests
Docs only: no test, no ablation (os-dev: docs are not ablated). The derived doc gates below are the verification.
Acceptance notes
origin/mainno page undercontent/docs/describes the SCIM group mapping that PR feat(plugin-auth): a SCIM group push grants the ordinary permission set its group mapping names #31443 landed (zero hits forset-group-mapping,sys_idp_group_mapping,groupDisplayNameoutside this diff). The link and back-link therefore use the existing SCIM Connection Credentials section, and the new SSO section carries the shared door's rules for both sources. Documenting the SCIM half's push-time behaviour is left out of this card's file surface.origin/main(PR docs(adr): ADR-0152 a SCIM group grants only an ordinary permission set, through a per-connection mapping row #31636 still open).register-sso-provider.tswritesoidcConfig.mappingas{ email, name }only and no SAML mapping, so the page sends operators to the vendor's/sso/registerforextraFields. Noted, not filed.skip-changeset. No package'sfiles[]shipscontent/docs/(grep of everypackage.jsonunderpackages/andapps/), so the diff publishes nothing.Verification
Head:
a7bcac5c(origin/main8c9260f7, which carries PR #31657, merged in; no rebase).Derived gates on the head
node scripts/pm/dispatch-gates.mjs --commands --repo objectstack-ai/objectstackona7bcac5cderived 45 commands (the same 45 as on the pre-merge headaf648c0e); all 45 ran with exit 0 after the spec, lint and client-react closure was rebuilt for the merged tree, and--ranreconciled: 45 derived, 45 run, 0 NOT-MEASURED, 0 UNRUN. Among themcheck:doc-anchors(956 internal fragment links resolve, the two new ones included),check:doc-authoring,check:doc-frontmatter,check:docs-single-h1,check:doc-route-spelling(advisory, population clean) andcheck:doc-security-posture.On the pre-merge head,
check:skill-examplesfirst exited 3 (PREREQUISITE NOT MET:client-reacthad no.d.ts); it ran green after that package was built, and green again on this head.Lint
No package is touched (two
.mdxpages only), so no package-scopedpnpm lintapplies.Merged behaviour read
sso-group-projection.ts,idp-group-mapping-admin.ts,scim-group-projection.ts,auth-manager-plugin-list.tsand #31657's changeset are byte-identical between #31657's last head70db6d06andorigin/main8c9260f7(git diff --statempty), so the page describes what landed.Generated by Claude Code