Skip to content

Commit 3bba1c0

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 981908a commit 3bba1c0

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

lib/net/imap.rb

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

807807
include MonitorMixin
808-
if defined?(OpenSSL::SSL)
809-
include OpenSSL
810-
include SSL
811-
end
812808

813809
# :call-seq:
814810
# Net::IMAP::SequenceSet(set = nil) -> SequenceSet

0 commit comments

Comments
 (0)