From: "drbrain (Eric Hodel)" <drbrain@...7.net>
Date: 2012-07-21T08:16:22+09:00
Subject: [ruby-core:46581] [ruby-trunk - Feature #6482] Add URI requested to Net::HTTP request and response objects


Issue #6482 has been updated by drbrain (Eric Hodel).

File net.http.request_response_uri.3.patch added

=begin
naruse (Yui NARUSE) wrote:
> drbrain (Eric Hodel) wrote:
> > This does not seem to match the current behavior of req['Host'] as it must be manually cleared in order to reuse the request with a different host.
> 
> Try following:
>   [���]
> 
> The host part of a URI for initialize seems to be the same thing of req['Host'].

I think I don't understand.  My patch uses the host part of URI for initialize to set req['Host']. Also, if you set req['Host'] the URI is updated correctly. Which server you connect to doesn't seem to matter.

I don't see the request Host header matching the connection host address with current net/http:

  $ svnversion
  36482
  $ cat test.rb 
  require 'net/http'
  req = Net::HTTP::Get.new '/'
  puts "req Host: #{req['Host']}"
  res = Net::HTTP.start 'redmine.ruby-lang.org' do |http|
    puts "con Host: #{http.address}"
    http.request req
  end
  puts "req Host: #{req['Host']}"
  res = Net::HTTP.start 'bugs.ruby-lang.org' do |http|
    puts "con Host: #{http.address}"
    http.request req
  end
  puts "req Host: #{req['Host']}"
  
  $ make runruby
  ./miniruby -I./lib -I. -I.ext/common  ./tool/runruby.rb --extout=.ext  -- --disable-gems ./test.rb
  req Host: 
  con Host: redmine.ruby-lang.org
  req Host: redmine.ruby-lang.org
  con Host: bugs.ruby-lang.org
  req Host: redmine.ruby-lang.org

My latest patch has identical behavior:

  $ patch -p0 < net.http.request_response_uri.3.patch 
  [���]
  $ make runruby
  ./miniruby -I./lib -I. -I.ext/common  ./tool/runruby.rb --extout=.ext  -- --disable-gems ./test.rb
  req Host: 
  con Host: redmine.ruby-lang.org
  req Host: redmine.ruby-lang.org
  con Host: bugs.ruby-lang.org
  req Host: redmine.ruby-lang.org

Identical test using URI instead of string path:

  $ cat test.rb 
  require 'net/http'
  u = URI("http://redmine.ruby-lang.org/")
  req = Net::HTTP::Get.new u
  puts "req Host: #{req['Host']}"
  puts "req URI:  #{req.uri}"
  res = Net::HTTP.start 'redmine.ruby-lang.org' do |http|
    puts "con Host: #{http.address}"
    http.request req
  end
  puts "req Host: #{req['Host']}"
  puts "req URI:  #{req.uri}"
  res = Net::HTTP.start 'bugs.ruby-lang.org' do |http|
    puts "con Host: #{http.address}"
    http.request req
  end
  puts "req Host: #{req['Host']}"
  puts "req URI:  #{req.uri}"
  
  $ make runruby
  ./miniruby -I./lib -I. -I.ext/common  ./tool/runruby.rb --extout=.ext  -- --disable-gems ./test.rb
  req Host: redmine.ruby-lang.org
  req URI:  http://redmine.ruby-lang.org/
  con Host: redmine.ruby-lang.org
  req Host: redmine.ruby-lang.org
  req URI:  http://redmine.ruby-lang.org/
  con Host: bugs.ruby-lang.org
  req Host: redmine.ruby-lang.org
  req URI:  http://redmine.ruby-lang.org/

=end

----------------------------------------
Feature #6482: Add URI requested to Net::HTTP request and response objects
https://bugs.ruby-lang.org/issues/6482#change-28241

Author: drbrain (Eric Hodel)
Status: Assigned
Priority: Normal
Assignee: naruse (Yui NARUSE)
Category: lib
Target version: 2.0.0


=begin
This patch adds the full URI requested to Net::HTTPRequest and Net::HTTPResponse.

The goal of this is to make it easier to handle Location, Refresh, meta-headers, and URIs in retrieved documents.  (While the HTTP RFC specifies the Location must be an absolute URI, not every server follows the RFC.)  In order to process redirect responses from bad servers or relative URIs in requested documents the user must create an object that contains both the requested URI and the response object to create absolute URIs.  This patch reduces the amount of boilerplate they are required to write.

Only the (({request_uri})) is used from the URI given when creating a request.  The URI is stored internally and updated with the host, port and scheme used to make the request at request time.  The URI is then copied to the response object for use by the user.

To preserve backwards compatibility the new behavior is optional.  This allows requests with invalid URI paths like (({Net::HTTP::Get.new '/f%'})) to continue to work.  Users of string paths will not be able to retrieve the requested URI.

This patch is for support of #5064
=end



-- 
http://bugs.ruby-lang.org/