Skip to content

Resolve "Mirroring for external CI/CD repositories should not store OAuth token"

Scott Hampton requested to merge 59053-no-oauth-for-cicd-github-fe into master

What does this MR do?

Oauth2 tokens are causing issues with mirroring repos, because it effectively limits the number of repos you can mirror. Personal Access Tokens do not have this problem.

This change removes the OAuth2 option from the import page, and only provides the personal access token form.

It also brings the form up to current design standards.

Normal import with OAuth configured

Before After
Screen_Shot_2019-07-31_at_9.09.36_AM Screen_Shot_2019-08-13_at_2.34.56_PM

Normal import without OAuth configured (and I'm an admin)

Before After
Screen_Shot_2019-07-31_at_9.09.36_AM Screen_Shot_2019-08-13_at_2.34.36_PM

CI/CD only import

Before After
Screen_Shot_2019-07-31_at_9.09.36_AM Screen_Shot_2019-08-13_at_2.35.26_PM

Related to #59053 (closed)

Does this MR meet the acceptance criteria?

Conformity

Performance and testing

Edited by Mayra Cabrera

Merge request reports