summaryrefslogtreecommitdiff
path: root/kernel.rb
diff options
context:
space:
mode:
authorJeremy Evans <code@jeremyevans.net>2022-01-04 10:22:37 -0800
committergit <svn-admin@ruby-lang.org>2022-02-03 05:10:21 +0900
commit7529c538915cd1765b0a362ca64e8f8ff439d7fc (patch)
tree8a7a3b2f0d49866a52fea9cde1650f9b2ddd3e19 /kernel.rb
parent3ca244a29a6a3b76d446de2ff45b09c1b5a1ac30 (diff)
downloadruby-7529c538915cd1765b0a362ca64e8f8ff439d7fc.tar.gz
[ruby/net-http] Do not set SNI hostname if connecting to IP address
RFC 6066, section 3, explicitly disallows the use of an IP address as an SNI server name. So check if the connection is being made to an IP address using the resolv regexps, and do not set an SNI hostname in that case. Recent changes to LibreSSL make it more strictly follow RFC 6066, resulting an s.hostname= raising an error if passed an IP address. When such verions of LibreSSL are used, this change not only fixes the net/http tests, it also fixes tests for webrick and open-uri, which both make SSL connections to 127.0.0.1 using net/http in their tests. Avoid warning in the openssl extension by unsetting @ssl_context.verify_hostname if connecting to an IP address. Make changes so that the post_connection_check still happens when connecting to an IP address, which is necessary to keep checking that the certificate returned includes the IP address, which one of the tests depends on. Revert the previous change that modified the regexp used for checking the error message. https://github.com/ruby/net-http/commit/fa68e64bee
Diffstat (limited to 'kernel.rb')
0 files changed, 0 insertions, 0 deletions