knu	2008-05-28 16:13:04 +0900 (Wed, 28 May 2008)

  New Revision: 16662

  Modified files:
    trunk/ChangeLog
    trunk/lib/webrick/httpservlet/cgihandler.rb

  Log:
    * lib/webrick/httpservlet/cgihandler.rb (WEBrick::HTTPServlet::CGIHandler#do_GET):
      Set the HTTP status code to 302 if a Location header field is
      present and the status code is not valid as a client
      redirection.  cf. RFC 3875 6.2.3, 6.2.4.


  http://svn.ruby-lang.org/cgi-bin/viewvc.cgi/trunk/ChangeLog?r1=16662&r2=16661&diff_format=u
  http://svn.ruby-lang.org/cgi-bin/viewvc.cgi/trunk/lib/webrick/httpservlet/cgihandler.rb?r1=16662&r2=16661&diff_format=u

Index: ChangeLog
===================================================================
--- ChangeLog	(revision 16661)
+++ ChangeLog	(revision 16662)
@@ -1,3 +1,10 @@
+Wed May 28 16:12:44 2008  Akinori MUSHA  <knu@i...>
+
+	* lib/webrick/httpservlet/cgihandler.rb (WEBrick::HTTPServlet::CGIHandler#do_GET):
+	  Set the HTTP status code to 302 if a Location header field is
+	  present and the status code is not valid as a client
+	  redirection.  cf. RFC 3875 6.2.3, 6.2.4.
+
 Wed May 28 15:53:52 2008  NAKAMURA Usaku  <usa@r...>
 
 	* enc/trans/japanese.c (to_SHIFT_JIS_EF_infos): typo.
Index: lib/webrick/httpservlet/cgihandler.rb
===================================================================
--- lib/webrick/httpservlet/cgihandler.rb	(revision 16661)
+++ lib/webrick/httpservlet/cgihandler.rb	(revision 16662)
@@ -87,6 +87,10 @@
             res.status = $1.to_i
             header.delete('status')
           end
+          if header.has_key?('location')
+            # RFC 3875 6.2.3, 6.2.4
+            res.status = 302 unless (300...400) === res.status
+          end
           if header.has_key?('set-cookie')
             header['set-cookie'].each{|k|
               res.cookies << Cookie.parse_set_cookie(k)

--
ML: ruby-changes@q...
Info: http://www.atdot.net/~ko1/quickml/