diff options
author | Toshi MARUYAMA <marutosijp2@yahoo.co.jp> | 2019-11-25 09:08:55 +0000 |
---|---|---|
committer | Toshi MARUYAMA <marutosijp2@yahoo.co.jp> | 2019-11-25 09:08:55 +0000 |
commit | bfeffd964b5ea626ada2d4fde676d6fe7cf9f854 (patch) | |
tree | 79df8c673551791f47db0b545edfa52618640df9 /app | |
parent | 0c341eb21d408dcbee33eccb13effc0bc0035bd1 (diff) | |
download | redmine-bfeffd964b5ea626ada2d4fde676d6fe7cf9f854.tar.gz redmine-bfeffd964b5ea626ada2d4fde676d6fe7cf9f854.zip |
cleanup: rubocop: fix Layout/MultilineHashBraceLayout and Layout/SpaceInsideBlockBraces in app/models/auth_source_ldap.rb
git-svn-id: http://svn.redmine.org/redmine/trunk@19293 e93f8b46-1217-0410-a6f0-8f06a7374b81
Diffstat (limited to 'app')
-rw-r--r-- | app/models/auth_source_ldap.rb | 14 |
1 files changed, 6 insertions, 8 deletions
diff --git a/app/models/auth_source_ldap.rb b/app/models/auth_source_ldap.rb index 3e1b7c10b..2a1791a66 100644 --- a/app/models/auth_source_ldap.rb +++ b/app/models/auth_source_ldap.rb @@ -70,8 +70,7 @@ class AuthSourceLdap < AuthSource def test_connection with_timeout do ldap_con = initialize_ldap_con(self.account, self.account_password) - ldap_con.open { } - + ldap_con.open {} if self.account.present? && !self.account.include?("$login") && self.account_password.present? ldap_auth = authenticate_dn(self.account, self.account_password) raise AuthSourceException.new(l(:error_ldap_bind_credentials)) if !ldap_auth @@ -177,20 +176,19 @@ class AuthSourceLdap < AuthSource end def initialize_ldap_con(ldap_user, ldap_password) - options = { :host => self.host, - :port => self.port - } + options = {:host => self.host, :port => self.port} if tls options[:encryption] = { :method => :simple_tls, # Always provide non-empty tls_options, to make sure, that all # OpenSSL::SSL::SSLContext::DEFAULT_PARAMS as well as the default cert # store are used. - :tls_options => { :verify_mode => verify_peer? ? OpenSSL::SSL::VERIFY_PEER : OpenSSL::SSL::VERIFY_NONE } + :tls_options => {:verify_mode => verify_peer? ? OpenSSL::SSL::VERIFY_PEER : OpenSSL::SSL::VERIFY_NONE} } end - - options.merge!(:auth => { :method => :simple, :username => ldap_user, :password => ldap_password }) unless ldap_user.blank? && ldap_password.blank? + unless ldap_user.blank? && ldap_password.blank? + options.merge!(:auth => {:method => :simple, :username => ldap_user, :password => ldap_password}) + end Net::LDAP.new options end |