summaryrefslogtreecommitdiffstats
path: root/app
diff options
context:
space:
mode:
authorToshi MARUYAMA <marutosijp2@yahoo.co.jp>2019-11-25 09:08:55 +0000
committerToshi MARUYAMA <marutosijp2@yahoo.co.jp>2019-11-25 09:08:55 +0000
commitbfeffd964b5ea626ada2d4fde676d6fe7cf9f854 (patch)
tree79df8c673551791f47db0b545edfa52618640df9 /app
parent0c341eb21d408dcbee33eccb13effc0bc0035bd1 (diff)
downloadredmine-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.rb14
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