* 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

2 files changed, 17 insertions, 15 deletions

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)