From bfeffd964b5ea626ada2d4fde676d6fe7cf9f854 Mon Sep 17 00:00:00 2001 From: Toshi MARUYAMA Date: Mon, 25 Nov 2019 09:08:55 +0000 Subject: [PATCH] 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 --- .rubocop_todo.yml | 1 - app/models/auth_source_ldap.rb | 14 ++++++-------- 2 files changed, 6 insertions(+), 9 deletions(-) diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml index 9e9f691db..660d9ee84 100644 --- a/.rubocop_todo.yml +++ b/.rubocop_todo.yml @@ -179,7 +179,6 @@ Layout/MultilineArrayBraceLayout: Layout/MultilineHashBraceLayout: Exclude: - 'app/controllers/context_menus_controller.rb' - - 'app/models/auth_source_ldap.rb' - 'lib/redmine/access_keys.rb' - 'lib/redmine/helpers/time_report.rb' - 'lib/redmine/scm/adapters/bazaar_adapter.rb' 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 -- 2.39.5