Skip to content

Commit acffb16

Browse files
committed
Add comment for TLS verification in 1.0 release
1 parent fd78782 commit acffb16

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

lib/net/ldap/connection.rb

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,9 @@ def self.wrap_with_ssl(io, tls_options = {})
4545
raise Net::LDAP::LdapError, "OpenSSL is unavailable" unless Net::LDAP::HasOpenSSL
4646

4747
ctx = OpenSSL::SSL::SSLContext.new
48+
49+
# By default, we do not verify certificates. For a 1.0 release, this should probably be changed at some point.
50+
# See discussion in https://github.com/ruby-ldap/ruby-net-ldap/pull/161
4851
ctx.set_params(tls_options) unless tls_options.empty?
4952

5053
conn = OpenSSL::SSL::SSLSocket.new(io, ctx)

0 commit comments

Comments
 (0)