Commit 0e9ba0a4 authored by Douwe Maan's avatar Douwe Maan

Add attributes to LDAP::Config.

parent c915e2c8
...@@ -97,6 +97,7 @@ if Settings.ldap['enabled'] || Rails.env.test? ...@@ -97,6 +97,7 @@ if Settings.ldap['enabled'] || Rails.env.test?
server['block_auto_created_users'] = false if server['block_auto_created_users'].nil? server['block_auto_created_users'] = false if server['block_auto_created_users'].nil?
server['allow_username_or_email_login'] = false if server['allow_username_or_email_login'].nil? server['allow_username_or_email_login'] = false if server['allow_username_or_email_login'].nil?
server['active_directory'] = true if server['active_directory'].nil? server['active_directory'] = true if server['active_directory'].nil?
server['attributes'] = {} if server['attributes'].nil?
server['provider_name'] ||= "ldap#{key}".downcase server['provider_name'] ||= "ldap#{key}".downcase
server['provider_class'] = OmniAuth::Utils.camelize(server['provider_name']) server['provider_class'] = OmniAuth::Utils.camelize(server['provider_name'])
end end
......
...@@ -84,6 +84,10 @@ module Gitlab ...@@ -84,6 +84,10 @@ module Gitlab
options['block_auto_created_users'] options['block_auto_created_users']
end end
def attributes
options['attributes']
end
protected protected
def base_config def base_config
Gitlab.config.ldap Gitlab.config.ldap
......
...@@ -68,12 +68,13 @@ module Gitlab ...@@ -68,12 +68,13 @@ module Gitlab
Gitlab::LDAP::Access.allowed?(gl_user) Gitlab::LDAP::Access.allowed?(gl_user)
end end
def ldap_config def ldap_config(provider = auth_hash.provider)
Gitlab::LDAP::Config.new(auth_hash.provider) Gitlab::LDAP::Config.new(provider)
end end
def auth_hash=(auth_hash) def auth_hash=(auth_hash)
@auth_hash = Gitlab::LDAP::AuthHash.new(auth_hash, ldap_config) config = ldap_config(auth_hash.provider)
@auth_hash = Gitlab::LDAP::AuthHash.new(auth_hash, config)
end end
end end
end end
......
...@@ -11,7 +11,7 @@ describe Gitlab::LDAP::User do ...@@ -11,7 +11,7 @@ describe Gitlab::LDAP::User do
} }
end end
let(:auth_hash) do let(:auth_hash) do
double(uid: 'my-uid', provider: 'ldapmain', info: double(info)) OmniAuth::AuthHash.new(uid: 'my-uid', provider: 'ldapmain', info: info)
end end
describe :changed? do describe :changed? do
......
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