Skip to content
Snippets Groups Projects

Fix OIDC account linking from profile

Merged Aboobacker MK requested to merge hotfix_oidc_account_linker into master
All threads resolved!

What does this MR do and why?

Fixes (Re)Connect to OIDC failed with "uninitialized ... (#411288 - closed)

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

  1. Setup omniauth provider for openid_connect
  2. Go-to -/profile/account
  3. Click on Connect open id connect
  4. Make sure that page is not showing any error

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Aboobacker MK

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
  • Imre Farkas approved this merge request

    approved this merge request

  • Imre Farkas requested review from @dstull and removed review request for @ifarkas

    requested review from @dstull and removed review request for @ifarkas

  • Aboobacker MK added 1 commit

    added 1 commit

    Compare with previous version

  • Contributor

    :warning: @tachyons-gitlab Some end-to-end (E2E) tests have been selected based on the stage label on this MR. If not run already, please run the e2e:package-and-test-ee job in the qa stage and review the results before merging this MR. (E2E tests are not run automatically on some MRs due to runner resource constraints.)

    If you would like to run all e2e tests, please apply the pipeline:run-all-e2e label and restart the pipeline.

    Once done, please apply the :white_check_mark: emoji on this comment.

    For any questions or help in reviewing the E2E test results, please reach out on the internal #quality Slack channel.

  • Doug Stull changed milestone to %16.1

    changed milestone to %16.1

  • Doug Stull approved this merge request

    approved this merge request

  • :wave: @dstull, thanks for approving this merge request.

    This is the first time the merge request is approved. To ensure full test coverage, a new pipeline will be started shortly.

    For more info, please refer to the following links:

  • Doug Stull resolved all threads

    resolved all threads

  • Doug Stull requested review from @ifarkas and removed review request for @dstull

    requested review from @ifarkas and removed review request for @dstull

  • Jessie Young resolved all threads

    resolved all threads

  • Jessie Young approved this merge request

    approved this merge request

  • Jessie Young requested review from @jessieay and removed review request for @ifarkas

    requested review from @jessieay and removed review request for @ifarkas

  • I've approved and set auto-merge

  • Jessie Young enabled an automatic merge when the pipeline for dc8a3c46 succeeds

    enabled an automatic merge when the pipeline for dc8a3c46 succeeds

  • merged

  • Jessie Young mentioned in commit bb908bbf

    mentioned in commit bb908bbf

  • added workflowstaging label and removed workflowcanary label

  • Please register or sign in to reply
    Loading