Fixes for use when OpenSSL is not loaded

Code in NIO::Monitor and NIO::Selector assumes OpenSSL is loaded, and raises if it isn't. Fix so OpenSSL isn't required. User must require it if processing SSLSockets.
socketry · Jul 4, 2020 · 6c6bf29 · 6c6bf29
1 parent 6c9ec5e
commit 6c6bf29
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/nio/monitor.rb b/lib/nio/monitor.rb
@@ -8,7 +8,7 @@ class Monitor
 
     # :nodoc:
     def initialize(io, interests, selector)
-      unless io.is_a? OpenSSL::SSL::SSLSocket
+      unless defined?(::OpenSSL) && io.is_a?(::OpenSSL::SSL::SSLSocket)
         unless io.is_a?(IO)
           if IO.respond_to? :try_convert
             io = IO.try_convert(io)

diff --git a/lib/nio/selector.rb b/lib/nio/selector.rb
@@ -44,7 +44,7 @@ def backend
     # * :w - is the IO writeable?
     # * :rw - is the IO either readable or writeable?
     def register(io, interest)
-      unless io.is_a? OpenSSL::SSL::SSLSocket
+      unless defined?(::OpenSSL) && io.is_a?(::OpenSSL::SSL::SSLSocket)
         io = IO.try_convert(io)
       end