From 6d69a240b86c10f128347a681c731c5cda972e22 Mon Sep 17 00:00:00 2001
From: gotoyuzo <gotoyuzo@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>
Date: Fri, 29 Apr 2005 22:10:56 +0000
Subject: * lib/webrick/cgi.rb: new methods WEBrick::CGI#[],
 WEBrick::CGI#logger   and WEBrick::CGI#config.  these are necessary to use an
 instance of   WEBrick::CGI as the first argument of HTTPServlet#get_instance.
   (suggested by Tatsuki Sugiura)

* lib/webrick/cgi.rb
  (WEBrick::CGI#initalize): set a dummy to @config[:ServerSoftware]
  if SERVER_SOFTWARE environment variable is not given.
  (WEBrick::CGI#start): req.path_info must be a String.
  (WEBrick::CGI::Socket#request_line): treat REQUEST_METHOD, PATH_INFO
  and SCRIPT_NAME to run in console.

* lib/webrick/httputils.rb (WEBrick::HTTPUtils.escape_path): should
  not use String#split("/"). it removes trailing empty path component.


git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@8393 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
---
 lib/webrick/cgi.rb       | 24 ++++++++++++------------
 lib/webrick/httputils.rb |  8 +++++---
 2 files changed, 17 insertions(+), 15 deletions(-)

(limited to 'lib/webrick')

diff --git a/lib/webrick/cgi.rb b/lib/webrick/cgi.rb
index eb99dab523..cab237ab04 100644
--- a/lib/webrick/cgi.rb
+++ b/lib/webrick/cgi.rb
@@ -16,6 +16,8 @@ module WEBrick
   class CGI
     CGIError = Class.new(StandardError)
 
+    attr_reader :config, :logger
+
     def initialize(*args)
       if defined?(MOD_RUBY)
         unless ENV.has_key?("GATEWAY_INTERFACE")
@@ -26,7 +28,7 @@ module WEBrick
         httpv = $1
       end
       @config = WEBrick::Config::HTTP.dup.update(
-        :ServerSoftware => ENV["SERVER_SOFTWARE"],
+        :ServerSoftware => ENV["SERVER_SOFTWARE"] || "null",
         :HTTPVersion    => HTTPVersion.new(httpv || "1.0"),
         :RunOnCGI       => true,   # to detect if it runs on CGI.
         :NPH            => false   # set true to run as NPH script.
@@ -39,6 +41,10 @@ module WEBrick
       @options = args
     end
 
+    def [](key)
+      @config[key]
+    end
+
     def start(env=ENV, stdin=$stdin, stdout=$stdout)
       sock = WEBrick::CGI::Socket.new(@config, env, stdin, stdout)
       req = HTTPRequest.new(@config)
@@ -58,12 +64,8 @@ module WEBrick
 
       begin
         req.parse(sock)
-        req.script_name = (env["SCRIPT_NAME"] || "").dup
-        if env["PATH_INFO"].nil? || env["PATH_INFO"].empty?
-          req.path_info = nil
-        else
-          req.path_info = env["PATH_INFO"].dup
-        end
+        req.script_name = (env["SCRIPT_NAME"] || File.expand_path($0)).dup
+        req.path_info = (env["PATH_INFO"] || "").dup
         req.user = env["REMOTE_USER"]
         res.request_method = req.request_method
         res.request_uri = req.request_uri
@@ -145,11 +147,9 @@ module WEBrick
       end
 
       def request_line
-        meth = @env["REQUEST_METHOD"]
-        url = @env["SCRIPT_NAME"].dup
-        if path_info = @env["PATH_INFO"]
-          url << path_info
-        end
+        meth = @env["REQUEST_METHOD"] || "GET"
+        url = (@env["SCRIPT_NAME"] || File.expand_path($0)).dup
+        url << @env["PATH_INFO"].to_s
         url = WEBrick::HTTPUtils.escape_path(url)
         if query_string = @env["QUERY_STRING"]
           unless query_string.empty?
diff --git a/lib/webrick/httputils.rb b/lib/webrick/httputils.rb
index e0855222f1..67bc281c5d 100644
--- a/lib/webrick/httputils.rb
+++ b/lib/webrick/httputils.rb
@@ -384,9 +384,11 @@ module WEBrick
     end
 
     def escape_path(str)
-      str.split("/").collect{|i|
-        _escape(i, UNESCAPED_PCHAR)
-      }.join("/")
+      result = ""
+      str.scan(%r{/([^/]*)}).each{|i|
+        result << "/" << _escape(i[0], UNESCAPED_PCHAR)
+      }
+      return result
     end
 
     def escape8bit(str)
-- 
cgit v1.2.3