* lib/webrick/httputils.rb (WEBrick::HTTPUtils.parse_range_header):

  fix regex for range-spec.

* lib/webrick/httpservlet/filehandler.rb
  (WEBrick::HTTPServlet::DefaultFileHandler#make_partial_content):
  multipart/byteranges response was broken.

* lib/xmlrpc/server.rb: refine example code.


git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@6761 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

2 files changed, 7 insertions, 3 deletions

diff --git a/lib/webrick/httpservlet/filehandler.rb b/lib/webrick/httpservlet/filehandler.rb
index 9e1a439b14..8525d7bb1b 100644
--- a/lib/webrick/httpservlet/filehandler.rb
+++ b/lib/webrick/httpservlet/filehandler.rb
@@ -72,13 +72,15 @@ module WEBrick
       def make_partial_content(req, res, filename, filesize)
         mtype = HTTPUtils::mime_type(filename, @config[:MimeTypes])
         unless ranges = HTTPUtils::parse_range_header(req['range'])
-          raise BadRequest, "Unrecognized range-spec: \"#{range}\""
+          raise HTTPStatus::BadRequest,
+            "Unrecognized range-spec: \"#{req['range']}\""
         end
         open(filename, "rb"){|io|
           if ranges.size > 1
+            time = Time.now
             boundary = "#{time.sec}_#{time.usec}_#{Process::pid}"
             body = ''
-            ranges.each{|r|
+            ranges.each{|range|
               first, last = prepare_range(range, filesize)
               next if first < 0
               io.pos = first
@@ -92,6 +94,8 @@ module WEBrick
             }
             raise HTTPStatus::RequestRangeNotSatisfiable if body.empty?
             body << "--" << boundary << "--" << CRLF
+            res["content-type"] = "multipart/byteranges; boundary=#{boundary}"
+            res.body = body
           elsif range = ranges[0]
             first, last = prepare_range(range, filesize)
             raise HTTPStatus::RequestRangeNotSatisfiable if first < 0
diff --git a/lib/webrick/httputils.rb b/lib/webrick/httputils.rb
index 79ea9d1f47..bae281ca1a 100644
--- a/lib/webrick/httputils.rb
+++ b/lib/webrick/httputils.rb
@@ -167,7 +167,7 @@ module WEBrick
           case range_spec
           when /^(\d+)-(\d+)/ then $1.to_i .. $2.to_i
           when /^(\d+)-/      then $1.to_i .. -1
-          when /^(\d+)/       then -($1.to_i) .. -1
+          when /^-(\d+)/      then -($1.to_i) .. -1
           else return nil
           end
         }