Merge pull request #383 from railsstarter/topic/fix_for_undefined_method_write_for_nilclass

HarlemSquirrel · web-flow · commit a37cc08616c0 · 2020-11-29T22:23:40.000-05:00
fix for undefined method for write exception
diff --git a/lib/net/ldap/connection.rb b/lib/net/ldap/connection.rb
@@ -690,7 +690,7 @@ def delete(args)
   #
   # Typically a TCPSocket, but can be a OpenSSL::SSL::SSLSocket
   def socket
-    return @conn if defined? @conn
+    return @conn if defined?(@conn) && !@conn.nil?
 
     # First refactoring uses the existing methods open_connection and
     # prepare_socket to set @conn. Next cleanup would centralize connection

Original file line number	Diff line number	Diff line change
`@@ -690,7 +690,7 @@ def delete(args)`
`690`	`690`	`#`
`691`	`691`	`# Typically a TCPSocket, but can be a OpenSSL::SSL::SSLSocket`
`692`	`692`	`def socket`
`693`		`- return @conn if defined? @conn`
	`693`	`+ return @conn if defined?(@conn) && !@conn.nil?`
`694`	`694`
`695`	`695`	`# First refactoring uses the existing methods open_connection and`
`696`	`696`	`# prepare_socket to set @conn. Next cleanup would centralize connection`