Skip to content
Snippets Groups Projects

Add auto_link_user Omniauth configuration setting

What does this MR do?

Adds an auto_link_user setting to the OmniAuth integrations, allowing current GitLab users to be automatically linked to their corresponding OmniAuth identities, as long as the email matches.

Set to false by default, and will apply the auto_link_saml_user behavior to all OmniAuth integrations.

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

Closes #24327 (closed)

Edited by Blair Lunceford

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
885 885 # (default: false)
886 886 auto_link_saml_user: false
887 887
888 # Allow users with existing accounts to login and auto link their account via OmniAuth
889 # login, without having to do a manual login first and manually add OmniAuth. Links on email.
890 # (default: false)
891 auto_link_user: false
  • Drew Blessing
  • Drew Blessing
  • Drew Blessing
  • mentioned in merge request omnibus-gitlab!4415 (merged)

  • added 1 commit

    • 6d98786c - Update spec and add changelog

    Compare with previous version

  • Blair Lunceford added 4540 commits

    added 4540 commits

    • 6d98786c...19368f19 - 4537 commits from branch master
    • 2162b619 - Add setting for auto_link_user
    • 2bfefafa - Update spec and add changelog
    • ea2b8cfd - Merge branch '24327-add-overarching-auto_link_user-omniauth-configuration' of...

    Compare with previous version

  • Drew Blessing approved this merge request

    approved this merge request

  • Blair Lunceford added 1167 commits

    added 1167 commits

    • ea2b8cfd...29518bb0 - 1163 commits from branch master
    • 8585afe2 - Add setting for auto_link_user
    • e4abf1cd - Update spec and add changelog
    • 508de501 - Merge branch '24327-add-overarching-auto_link_user-omniauth-configuration' of...
    • 59d1aadd - Applied suggestion

    Compare with previous version

  • Blair Lunceford mentioned in commit 26374d2f

    mentioned in commit 26374d2f

  • Blair Lunceford added 7 commits

    added 7 commits

    • ec74d212 - Adds job token auth to nuget
    • 8d8046af - Fix EachBatch call in license management migration
    • baf8a271 - Add docs about naming project bot users, and not using seat
    • 0a1df73e - Documentation changes for Expired PAT email
    • 3482dead - Add a section for target branches in GitLab Security
    • 26374d2f - Add auto_link_user OmniAuth setting
    • 66f03589 - Merge branch '24327-add-overarching-auto_link_user-omniauth-configuration' of...

    Compare with previous version

  • Blair Lunceford mentioned in commit ca7d9129

    mentioned in commit ca7d9129

  • Blair Lunceford added 2 commits

    added 2 commits

    • ca7d9129 - Add auto_link_user OmniAuth setting
    • 991bb908 - Merge branch '24327-add-overarching-auto_link_user-omniauth-configuration' of...

    Compare with previous version

  • Blair Lunceford mentioned in commit 0ef8d8e3

    mentioned in commit 0ef8d8e3

  • added 1 commit

    • 0ef8d8e3 - Add auto_link_user OmniAuth setting

    Compare with previous version

  • James Fargher
  • James Fargher
  • James Fargher
  • James Fargher
  • Blair Lunceford mentioned in commit 6918999f

    mentioned in commit 6918999f

  • added 1 commit

    • 6918999f - Add auto_link_user OmniAuth setting

    Compare with previous version

  • Mike Jang
  • Mike Jang approved this merge request

    approved this merge request

  • unassigned @mjang1

  • Blair Lunceford mentioned in issue #238626

    mentioned in issue #238626

  • added 1 commit

    • f8cf5936 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • James Fargher resolved all threads

    resolved all threads

  • James Fargher resolved all threads

    resolved all threads

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading