Skip to content

SCIM jit edits#24600

Open
akristen wants to merge 5 commits intodocker:mainfrom
akristen:scim-jit-edits
Open

SCIM jit edits#24600
akristen wants to merge 5 commits intodocker:mainfrom
akristen:scim-jit-edits

Conversation

@akristen
Copy link
Copy Markdown
Contributor

@akristen akristen commented Apr 1, 2026

Updating Provision section, part of freshness updates. Will add details later. 🖖

@akristen akristen self-assigned this Apr 1, 2026
@akristen akristen added the area/admin Relates to Docker Admin label Apr 1, 2026
@netlify
Copy link
Copy Markdown

netlify bot commented Apr 1, 2026

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit c03ffc8
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/69cd78fabe2913000805bb07
😎 Deploy Preview https://deploy-preview-24600--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

akristen added 3 commits April 1, 2026 10:59
…ce it contained two topics; moved domain management into provision since sso has its own instructions in setup
@akristen
Copy link
Copy Markdown
Contributor Author

akristen commented Apr 1, 2026

/review

@docker-agent
Copy link
Copy Markdown
Contributor

docker-agent bot commented Apr 1, 2026

PR Review Failed — The review agent encountered an error and could not complete the review. View logs.

Comment on lines 9 to 11
aliases:
- /security/for-admins/single-sign-on/troubleshoot/
- /security/troubleshoot/troubleshoot-sso/
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.

let's add a redirect for /enterprise/troubleshoot/troubleshoot-sso/

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

i've been fighting for my life against these redirects lmao. thank you!

Comment on lines +51 to +59
> [!NOTE]
>
> SCIM only manages users provisioned through your identity provider after
> SCIM is enabled. It cannot remove users who were manually added to your Docker
> organization before SCIM was set up.
>
> To remove those users, delete them manually from your Docker organization.
> For more information, see
> [Manage organization members](/manuals/admin/organization/members.md).
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.

question: I am not really familiar with SCIM at all, but, does this also apply if you (for whatever reason) manually add a user to an organization (they are not provisioned through SCIM). Or only if they were manually added "before SCIM was set up"?

Suggested change
> [!NOTE]
>
> SCIM only manages users provisioned through your identity provider after
> SCIM is enabled. It cannot remove users who were manually added to your Docker
> organization before SCIM was set up.
>
> To remove those users, delete them manually from your Docker organization.
> For more information, see
> [Manage organization members](/manuals/admin/organization/members.md).
> [!NOTE]
>
> SCIM only manages users provisioned through your identity provider.
> It cannot remove users who were manually added to your Docker organization.
>
> To remove manually added users, delete them manually from your Docker
> organization. For more information, see
> [Manage organization members](/manuals/admin/organization/members.md).

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good question. Do you mean users who are manually added after some method of provisioning is enabled in an org (auto/jit/scim)? Let me loop into @madisonsilva for a for sure answer -- i don't want to make assumptions, since the differences between the 3 provisioning methods are a bit subtle.

@dvdksn
Copy link
Copy Markdown
Contributor

dvdksn commented Apr 2, 2026

/review

Copy link
Copy Markdown
Contributor

@docker-agent docker-agent bot left a comment

Choose a reason for hiding this comment

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

Assessment: 🔴 CRITICAL

This PR reorganizes provisioning documentation with mostly good changes, but there is one critical missing redirect that will cause broken links.

---
title: Troubleshoot single sign-on
linkTitle: Troubleshoot SSO
linkTitle: Troubleshoot
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.

Missing redirect for moved page

This file is being moved from content/manuals/enterprise/troubleshoot/troubleshoot-sso.md to content/manuals/enterprise/security/single-sign-on/FAQs/troubleshoot-sso.md.

The old URL was /enterprise/troubleshoot/troubleshoot-sso/ (after Hugo strips the /manuals prefix).
The new URL is /enterprise/security/single-sign-on/faqs/troubleshoot-sso/.

The aliases section currently has:

  • /security/for-admins/single-sign-on/troubleshoot/
  • /security/troubleshoot/troubleshoot-sso/

But it's missing an alias for the old path /enterprise/troubleshoot/troubleshoot-sso/. This will cause broken links for anyone who bookmarked or linked to the old URL.

Fix: Add /enterprise/troubleshoot/troubleshoot-sso/ to the aliases array:

aliases:
    - /security/for-admins/single-sign-on/troubleshoot/
    - /security/troubleshoot/troubleshoot-sso/
    - /enterprise/troubleshoot/troubleshoot-sso/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/admin Relates to Docker Admin

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants