Author: markt
Date: Wed Aug 12 11:36:22 2009
New Revision: 803457

URL: http://svn.apache.org/viewvc?rev=803457&view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=46907
Debug logging should not cause input stream to be swallowed
http://svn.apache.org/viewvc?view=rev&revision=797168

Modified:
    tomcat/tc5.5.x/trunk/STATUS.txt
    tomcat/tc5.5.x/trunk/container/webapps/docs/changelog.xml
    
tomcat/tc5.5.x/trunk/jasper/src/share/org/apache/jasper/servlet/JspServlet.java

Modified: tomcat/tc5.5.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/STATUS.txt?rev=803457&r1=803456&r2=803457&view=diff
==============================================================================
--- tomcat/tc5.5.x/trunk/STATUS.txt (original)
+++ tomcat/tc5.5.x/trunk/STATUS.txt Wed Aug 12 11:36:22 2009
@@ -77,12 +77,6 @@
   -1: 
   rjung: minus the generics
 
-* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=46907
-  Debug logging should not cause input stream to be swallowed
-  http://svn.apache.org/viewvc?view=rev&revision=797168
-  +1: markt, kkolinko, rjung
-  -1: 
-
 * https://issues.apache.org/bugzilla/show_bug.cgi?id=47566
   Update to NSIS 2.45
   http://svn.apache.org/viewvc?rev=797596&view=rev

Modified: tomcat/tc5.5.x/trunk/container/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/container/webapps/docs/changelog.xml?rev=803457&r1=803456&r2=803457&view=diff
==============================================================================
--- tomcat/tc5.5.x/trunk/container/webapps/docs/changelog.xml (original)
+++ tomcat/tc5.5.x/trunk/container/webapps/docs/changelog.xml Wed Aug 12 
11:36:22 2009
@@ -69,6 +69,10 @@
       <fix>
         <bug>41661</bug>: Fix thread safety issue in JspConfig.init() (markt)
       </fix>
+      <fix>
+        <bug>46907</bug>: Don't swallow input stream when debug logging is
+        enabled. (markt)
+      </fix>
     </changelog>
   </subsection>
   <subsection name="Cluster">

Modified: 
tomcat/tc5.5.x/trunk/jasper/src/share/org/apache/jasper/servlet/JspServlet.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/jasper/src/share/org/apache/jasper/servlet/JspServlet.java?rev=803457&r1=803456&r2=803457&view=diff
==============================================================================
--- 
tomcat/tc5.5.x/trunk/jasper/src/share/org/apache/jasper/servlet/JspServlet.java 
(original)
+++ 
tomcat/tc5.5.x/trunk/jasper/src/share/org/apache/jasper/servlet/JspServlet.java 
Wed Aug 12 11:36:22 2009
@@ -19,7 +19,6 @@
 
 import java.io.IOException;
 import java.lang.reflect.Constructor;
-import java.util.Enumeration;
 
 import javax.servlet.ServletConfig;
 import javax.servlet.ServletContext;
@@ -251,13 +250,6 @@
             log.debug("\t        RealPath: " + context.getRealPath(jspUri));
             log.debug("\t      RequestURI: " + request.getRequestURI());
             log.debug("\t     QueryString: " + request.getQueryString());
-            log.debug("\t  Request Params: ");
-            Enumeration e = request.getParameterNames();
-            while (e.hasMoreElements()) {
-                String name = (String) e.nextElement();
-                log.debug("\t\t " + name + " = " 
-                          + request.getParameter(name));
-            }
         }
 
         try {



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to