Skip to content

Refactoring Gitlab::Geo::OauthSession

Douglas Barbosa Alexandre requested to merge da-refactor-geo-oauth-session into master

What does this MR do?

The Gitlab::Geo::OauthSession has a lot of responsibilities, this MR extracts each responsability into a proper class:

  • GitLab::Geo::Oauth::Session
  • GitLab::Geo::Oauth::LoginState
  • GitLab::Geo::Oauth::LogoutState
  • GitLab::Geo::Oauth::LogoutToken
  • GitLab::ReturnToLocation

What are the relevant issue numbers?

Closes https://gitlab.com/gitlab-org/gitlab-ee/issues/8422

Does this MR meet the acceptance criteria?

Edited by Douglas Barbosa Alexandre

Merge request reports