From 108218a89c663af1c14cc6ff376464e71639f974 Mon Sep 17 00:00:00 2001
From: Evan Weaver <eweaver@twitter.com>
Date: Sat, 31 Jan 2009 22:09:10 -0800
Subject: Clean up some logging.

---
 ext/http11_java/org/jruby/mongrel/Http11.java |  4 ++--
 lib/mongrel.rb                                | 12 ++++++------
 lib/mongrel/http_request.rb                   |  5 ++---
 3 files changed, 10 insertions(+), 11 deletions(-)

diff --git a/ext/http11_java/org/jruby/mongrel/Http11.java b/ext/http11_java/org/jruby/mongrel/Http11.java
index 3dbfdd6..4ab5c7f 100644
--- a/ext/http11_java/org/jruby/mongrel/Http11.java
+++ b/ext/http11_java/org/jruby/mongrel/Http11.java
@@ -213,9 +213,9 @@ public class Http11 extends RubyObject {
                     }
                 }
 
-                req.setInstanceVariable("@http_body", RubyString.newString(runtime, new ByteList(hp.parser.buffer, at, length)));
+                req.aset(runtime.newString("http_body"),RubyString.newString(runtime, new ByteList(hp.parser.buffer, at, length)));
                 req.aset(runtime.newString("SERVER_PROTOCOL"),runtime.newString("HTTP/1.1"));
-                req.aset(runtime.newString("SERVER_SOFTWARE"),runtime.newString("Mongrel 1.2.0"));
+                req.aset(runtime.newString("SERVER_SOFTWARE"),runtime.newString("Mongrel 2.0"));
             }
         };
 
diff --git a/lib/mongrel.rb b/lib/mongrel.rb
index 39659a2..515ae2e 100644
--- a/lib/mongrel.rb
+++ b/lib/mongrel.rb
@@ -115,7 +115,7 @@ module Mongrel
             end
 
             raise "No REQUEST PATH" if !params[Const::REQUEST_PATH]
-
+ 
             params[Const::PATH_INFO] = params[Const::REQUEST_PATH]
             params[Const::SCRIPT_NAME] = Const::SLASH
 
@@ -149,12 +149,12 @@ module Mongrel
       rescue EOFError,Errno::ECONNRESET,Errno::EPIPE,Errno::EINVAL,Errno::EBADF
         client.close rescue nil
       rescue HttpParserError => e
-        logger.error "#HTTP parse error, malformed request (#{params[Const::HTTP_X_FORWARDED_FOR] || client.peeraddr.last}): #{e.inspect}"
-        logger.error "#REQUEST DATA: #{data.inspect}\n---\nPARAMS: #{params.inspect}\n---\n"
+        logger.error "HTTP parse error, malformed request (#{params[Const::HTTP_X_FORWARDED_FOR] || client.peeraddr.last}): #{e.inspect}"
+        logger.error "REQUEST DATA: #{data.inspect}\n---\nPARAMS: #{params.inspect}\n---\n"
       rescue Errno::EMFILE
         reap_dead_workers('too many files')
       rescue Object => e
-        logger.error "#Read error: #{e.inspect}"
+        logger.error "Read error: #{e.inspect}"
         logger.error e.backtrace.join("\n")
       ensure
         begin
@@ -162,7 +162,7 @@ module Mongrel
         rescue IOError
           # Already closed
         rescue Object => e
-          logger.error "#Client error: #{e.inspect}"
+          logger.error "Client error: #{e.inspect}"
           logger.error e.backtrace.join("\n")
         end
         request.body.close! if request and request.body.class == Tempfile
@@ -175,7 +175,7 @@ module Mongrel
     # after the reap is done.  It only runs if there are workers to reap.
     def reap_dead_workers(reason='unknown')
       if @workers.list.length > 0
-        logger.info "#Reaping #{@workers.list.length} threads for slow workers because of '#{reason}'"
+        logger.info "Reaping #{@workers.list.length} threads for slow workers because of '#{reason}'"
         error_msg = "Mongrel timed out this thread: #{reason}"
         mark = Time.now
         @workers.list.each do |worker|
diff --git a/lib/mongrel/http_request.rb b/lib/mongrel/http_request.rb
index f76e0bb..28496cd 100644
--- a/lib/mongrel/http_request.rb
+++ b/lib/mongrel/http_request.rb
@@ -42,9 +42,8 @@ module Mongrel
       @body.rewind if @body
     end
 
-    # returns an environment which is rackable
-    # http://rack.rubyforge.org/doc/files/SPEC.html
-    # copied directly from racks mongrel handler
+    # Returns an environment which is rackable: http://rack.rubyforge.org/doc/files/SPEC.html
+    # Copied directly from Rack's old Mongrel handler.
     def env
       env = params.clone
       env.delete "HTTP_CONTENT_TYPE"
-- 
cgit v1.2.3-24-ge0c7