Net::HTTP.get_response can receive URI as string

Author: kyanagi

lib/net/http.rb

@@ -796,6 +796,12 @@ def HTTP.get_print(uri_or_host, path_or_headers = nil, port = nil)
     #   headers = {'Content-type' => 'application/json; charset=UTF-8'}
     #   Net::HTTP.get(uri, headers)
     #
+    # Alternatively, +uri+ may be a String:
+    #
+    #   uri = 'https://jsonplaceholder.typicode.com/todos/1'
+    #   headers = {'Content-type' => 'application/json; charset=UTF-8'}
+    #   Net::HTTP.get(uri, headers)
+    #
     # Related:
     #
     # - Net::HTTP::Get: request class for \HTTP method +GET+.
@@ -820,6 +826,7 @@ def HTTP.get_response(uri_or_host, path_or_headers = nil, port = nil, &block)
         }
       else
         uri = uri_or_host
+        uri = URI(uri) if uri.is_a?(String)
         headers = path_or_headers
         start(uri.hostname, uri.port,
               :use_ssl => uri.scheme == 'https') {|http|

test/net/http/test_http.rb

@@ -303,6 +303,13 @@ def test_s_get
     assert_equal $test_net_http_data,
         Net::HTTP.get(config('host'), '/', config('port'))
 
+    assert_equal $test_net_http_data,
+        Net::HTTP.get("http://#{config('host')}:#{config('port')}")
+
+    assert_equal $test_net_http_data, Net::HTTP.get(
+      "http://#{config('host')}:#{config('port')}", "Accept" => "text/plain"
+    )
+
     assert_equal $test_net_http_data, Net::HTTP.get(
       URI.parse("http://#{config('host')}:#{config('port')}")
     )
@@ -311,7 +318,23 @@ def test_s_get
     )
   end
 
-  def test_s_get_response
+  def test_s_get_response_with_host
+    res = Net::HTTP.get_response(config('host'), '/', config('port'))
+    assert_equal "application/octet-stream", res["Content-Type"]
+    assert_equal $test_net_http_data, res.body
+  end
+
+  def test_s_get_response_with_uri_string
+    res = Net::HTTP.get_response("http://#{config('host')}:#{config('port')}")
+    assert_equal "application/octet-stream", res["Content-Type"]
+    assert_equal $test_net_http_data, res.body
+
+    res = Net::HTTP.get_response("http://#{config('host')}:#{config('port')}", "Accept" => "text/plain")
+    assert_equal "text/plain", res["Content-Type"]
+    assert_equal $test_net_http_data, res.body
+  end
+
+  def test_s_get_response_with_uri
     res = Net::HTTP.get_response(
       URI.parse("http://#{config('host')}:#{config('port')}")
     )