Skip to content

Commit ed743d2

Browse files
committed
💥 Do not mixin OpenSSL::SSL module
`Net::IMAP` has mixed in `OpenSSL::SSL` since it first added TLS support. But that isn't very helpful, and it clutters up the namespace with irrelevant constants. I doubt anyone is actually _using_ OpenSSL constants from Net::IMAP. I hope not. Nevertheless, this is a breaking change.
1 parent 8797c5d commit ed743d2

1 file changed

Lines changed: 0 additions & 4 deletions

File tree

lib/net/imap.rb

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -804,10 +804,6 @@ class IMAP < Protocol
804804
autoload :StringPrep, "#{dir}/stringprep"
805805

806806
include MonitorMixin
807-
if defined?(OpenSSL::SSL)
808-
include OpenSSL
809-
include SSL
810-
end
811807

812808
# Returns the global Config object
813809
def self.config; Config.global end

0 commit comments

Comments
 (0)