Skip to content
This repository was archived by the owner on Dec 13, 2023. It is now read-only.

Conversation

nerpaula
Copy link
Contributor

@nerpaula nerpaula commented Feb 7, 2023

Update ArangoGraph Single-Sign On (SSO) feature:

  • separate sections for SAML 2.0 and SCIM
  • step-by-step procedures on how to configure SAML 2.0 and SCIM using Okta

preview: https://deploy-preview-1262--zealous-morse-14392b.netlify.app/docs/3.10/arangograph/single-sign-on.html

@cla-bot cla-bot bot added the cla-signed label Feb 7, 2023
@nerpaula nerpaula self-assigned this Feb 7, 2023
Copy link
Contributor

@mattcarabine mattcarabine left a comment

Choose a reason for hiding this comment

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

Left some individual comments but the main gist is that we need to change the way that this SSO stuff flows.

We need to heavily de-emphasise SCIM and more cleanly separate SCIM and SAML. It's fine that all of the SAML configuration steps are a pre-requisite to enable SCIM, but right now it looks like they're fully combined. I'd suggest something like a completely different page for SCIM so that we don't mix them.

e.g. a page that goes through all the steps of configuring SAML SSO, then a "If you're interested in SCIM, you can also look here" as a follow-on.

As it reads today, everybody will be enabling SCIM and we do not want that.

Copy link
Contributor

@mattcarabine mattcarabine left a comment

Choose a reason for hiding this comment

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

Have requested one small change, but this looks so much better @nerpaula, flows very nicely, thank you very much!

@nerpaula nerpaula merged commit eb6a4f9 into main Feb 16, 2023
@nerpaula nerpaula deleted the DOC-329 branch February 16, 2023 06:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

5 participants