Skip to content

Geo: Fix OAuth authentication with relative URL used on secondary

Valery Sizov requested to merge oauth-geo-fix into master

What does this MR do?

When we build primary's URL for Oauth we don't have to use URL helpers on secondary node as they can be misleading, for example, because of the relative URL or protocol or something else. Moreover, this change makes the code look consistent.

Closes #11261 (closed)

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 Valery Sizov

Merge request reports

Loading