FIX: my sso_overrides_username assumed username was passed

This commit is contained in:
Michael Brown 2018-03-13 18:29:11 -04:00
parent 6b13fbccc2
commit 1b5549df58

View File

@ -181,10 +181,10 @@ class DiscourseSingleSignOn < SingleSignOn
user.active = false if require_activation
end
if SiteSetting.sso_overrides_username?
if SiteSetting.sso_overrides_username? && username.present?
if user.username.downcase == username.downcase
user.username = username # there may be a change of case
elsif user.username != username && username.present?
elsif user.username != username
user.username = UserNameSuggester.suggest(username || name || email, user.username)
end
end