Commit 5340ff29 authored by Tiago Botelho's avatar Tiago Botelho

Fixes CE upstream conflict in Gitlab::SAML::User.

parent 237a9a35
...@@ -17,10 +17,9 @@ module Gitlab ...@@ -17,10 +17,9 @@ module Gitlab
user ||= find_or_build_ldap_user if auto_link_ldap_user? user ||= find_or_build_ldap_user if auto_link_ldap_user?
user ||= build_new_user if signup_enabled? user ||= build_new_user if signup_enabled?
if external_users_enabled? && user if user
# Check if there is overlap between the user's groups and the external groups user.external = !(auth_hash.groups & Gitlab::Saml::Config.external_groups).empty? if external_users_enabled?
# setting then set user as external or internal. user.admin = !(auth_hash.groups & Gitlab::Saml::Config.admin_groups).empty? if admin_groups_enabled?
user.external = !(auth_hash.groups & Gitlab::Saml::Config.external_groups).empty?
end end
user user
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment