diff options
author | Jeremy Evans <code@jeremyevans.net> | 2020-07-29 12:34:07 -0700 |
---|---|---|
committer | Hiroshi SHIBATA <hsbt@ruby-lang.org> | 2020-08-03 18:55:44 +0900 |
commit | 20eb9e98b65352422fbb1197eb8699bfcb12d5ef (patch) | |
tree | 49c1ea447ecee11a7105e9a3bbb34076e49c5e7a /test | |
parent | e732d376affbf0fea8910ece53338cce57c25ad4 (diff) | |
download | ruby-20eb9e98b65352422fbb1197eb8699bfcb12d5ef.tar.gz |
[ruby/net-http] Switch invalid server name format
invalid_servername is not a valid name in an SSL request due to
the use of the underscore, and LibreSSL 3.2.0 will raise an
exception for this. These tests are not testing the allowed
characters in the server name, but how net/http handles cases where
the server name provided does not match the IP address you are
trying to connect to, so I think it's better to just modify the
tests to use a correct format.
While here, fix a typo in a test name, and use better code in the
ensure block so the same test doesn't issue both a failure and an
error.
https://github.com/ruby/net-http/commit/0e8dc91120
Diffstat (limited to 'test')
-rw-r--r-- | test/net/http/test_https.rb | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/test/net/http/test_https.rb b/test/net/http/test_https.rb index 660104d048..7660ef686c 100644 --- a/test/net/http/test_https.rb +++ b/test/net/http/test_https.rb @@ -108,7 +108,7 @@ class TestNetHTTPS < Test::Unit::TestCase def test_get_SNI_failure TestNetHTTPUtils.clean_http_proxy_env do - http = Net::HTTP.new("invalid_servername", config("port")) + http = Net::HTTP.new("invalidservername", config("port")) http.ipaddr = config('host') http.use_ssl = true http.cert_store = TEST_STORE @@ -204,22 +204,22 @@ class TestNetHTTPS < Test::Unit::TestCase skip $! end - def test_skip_hostname_verfiction + def test_skip_hostname_verification TestNetHTTPUtils.clean_http_proxy_env do - http = Net::HTTP.new('invalid_servername', config('port')) + http = Net::HTTP.new('invalidservername', config('port')) http.ipaddr = config('host') http.use_ssl = true http.cert_store = TEST_STORE http.verify_hostname = false assert_nothing_raised { http.start } ensure - http&.finish + http.finish if http&.started? end end def test_fail_if_verify_hostname_is_true TestNetHTTPUtils.clean_http_proxy_env do - http = Net::HTTP.new('invalid_servername', config('port')) + http = Net::HTTP.new('invalidservername', config('port')) http.ipaddr = config('host') http.use_ssl = true http.cert_store = TEST_STORE |