Skip to content

Try to use username given by omniauth provider

username-removed-36741 requested to merge (removed):try-username-first-before-using-mail-part into master

What does this MR do?
Tries to use the username given by an oauth provider

Are there points in the code the reviewer needs to double check?
Perhaps use another condition than empty.

Why was this MR needed?
If an oauth provider is used as external provider it always takes the part before the @ as username regardless if an username is given by the provider or not. Most people are expecting that the username given by the provider is used.

See #326 (closed)

Merge request reports