Bug #9750: OpenSSL::SSL::SSLServer can not accept a Socket as it's first parameter

• Status: Closed
• Assignee: -
• Target version: 2.2.0
• ruby -v: ruby 2.2.0dev (2014-04-16 trunk 45603) [x86_64-linux]
• Backport: 2.0.0: DONE, 2.1: DONE

[sstelfox (Sam Stelfox)]

This could potentially be an issue with a lack of documentation, in which case this may be more appropriate as a feature request.

The first parameter of OpenSSL::SSL::SSLServer#initialize takes a TCPServer without any issue, however, when provided with an instance of Socket that has been created with Socket::SOCK_STREAM (so a standard TCP socket) it breaks with a "wrong argument type Array (expected File)" when attempting to accept a new client. I tracked this down to the difference in return values from TCPServer#accept (returns just a file descriptor) and Socket#accept (which returns an array of file descriptor and address info) and I believe it would be fairly easy to adjust the SSLServer to be able to handle both.

I've attached some sample standalone code that demonstrates the issue and has a work around based on a subclass'd version of SSLServer that can handle both TCPServer and Socket objects. The important lines are 45-58 as well as 85-86. Everything else is setup and support too get those running or are comments. When running without modification it will demonstrate the error. By swapping the comments on 85-86 you can use the adjusted version of SSLServer#accept.

Please let me know if there are any questions, or concerns. I'm happy to help in anyway around this issue.

[akr (Akira Tanaka)]

• Status changed from Open to Closed
• % Done changed from 0 to 100

Applied in changeset r45863.

---

• ext/openssl/lib/openssl/ssl.rb (OpenSSL::SSL::SSLServer#accept):
  Consider Socket#accept as well as TCPServer#accept.
  Reported by Sam Stelfox. [ruby-core:62064] [Bug #9750]

[nagachika (Tomoyuki Chikanaga)]

• Backport changed from 2.0.0: UNKNOWN, 2.1: UNKNOWN to 2.0.0: REQUIRED, 2.1: REQUIRED

I think this introduces no incompatibility and should be backported.

[usa (Usaku NAKAMURA)]

• Backport changed from 2.0.0: REQUIRED, 2.1: REQUIRED to 2.0.0: DONE, 2.1: REQUIRED

backported r45863 and r45871 into ruby_2_0_0 at r46579.

[nagachika (Tomoyuki Chikanaga)]

• Backport changed from 2.0.0: DONE, 2.1: REQUIRED to 2.0.0: DONE, 2.1: DONE

r45863 and r45871 were backported into ruby_2_1 branch at r46613.