Skip to content

Full SCIM Identities Support

Drew Blessing requested to merge dblessing-full-scim-identities into master

What does this MR do?

Second part of changes for #208940 (closed)

The previous MR (!26124 (merged)) added the feature flag plus the GET endpoint support.

This change adds support for SCIM Identities in the SCIM API endpoints for POST (create SCIM user), PATCH (update/deprovision), and DELETE (deprovision).

After some testing on .com a final MR will come to enable scim_identities by default and migrate (copy) all SAML identities to SCIM identities.

There's no changelog since there are no migrations and this isn't yet a user-facing change.

Next steps

To give you a picture of what comes after this MR so you have more context:

  1. Once this is deployed to GitLab.com and staging, enable it for a test group (i.e. gitlab-silver) and make sure things work as expected.
  2. Merge MR !26940 (merged) (WIP) that migrates existing SAML/SCIM combined identities to the standalone SCIM Identity as well as defaults the feature to on.
  3. Add Okta integration on top !25649 (merged)

Screenshots

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Drew Blessing

Merge request reports