diff options
author | Jacob Vosmaer <contact@jacobvosmaer.nl> | 2014-10-23 22:57:16 +0200 |
---|---|---|
committer | Valery Sizov <valery@gitlab.com> | 2014-10-24 12:53:32 +0300 |
commit | 16a10eb1cd4a5447da9d50b1eba25f020dc8f6b7 (patch) | |
tree | 83c5a6a7569b4189bbc3b59f9001a2759d769fb8 /lib/gitlab/ldap/config.rb | |
parent | 5f7906e1635baa1aca12527ac9d9f8e84323e95d (diff) | |
download | gitlab-ce-16a10eb1cd4a5447da9d50b1eba25f020dc8f6b7.tar.gz |
Fix LDAP config lookup for provider 'ldap'
Diffstat (limited to 'lib/gitlab/ldap/config.rb')
-rw-r--r-- | lib/gitlab/ldap/config.rb | 27 |
1 files changed, 16 insertions, 11 deletions
diff --git a/lib/gitlab/ldap/config.rb b/lib/gitlab/ldap/config.rb index d41bfba9b0f..0cb24d0ccc1 100644 --- a/lib/gitlab/ldap/config.rb +++ b/lib/gitlab/ldap/config.rb @@ -16,10 +16,23 @@ module Gitlab servers.map {|server| server['provider_name'] } end + def self.valid_provider?(provider) + providers.include?(provider) + end + + def self.invalid_provider(provider) + raise "Unknown provider (#{provider}). Available providers: #{providers}" + end + def initialize(provider) - @provider = provider - invalid_provider unless valid_provider? - @options = config_for(provider) + if self.class.valid_provider?(provider) + @provider = provider + elsif provider == 'ldap' + @provider = self.class.providers.first + else + self.class.invalid_provider(provider) + end + @options = config_for(@provider) # Use @provider, not provider end def enabled? @@ -89,14 +102,6 @@ module Gitlab end end - def valid_provider? - self.class.providers.include?(provider) - end - - def invalid_provider - raise "Unknown provider (#{provider}). Available providers: #{self.class.providers}" - end - def auth_options { auth: { |