tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From kkoli...@apache.org
Subject svn commit: r1231773 - /tomcat/tc6.0.x/trunk/STATUS.txt
Date Sun, 15 Jan 2012 22:14:02 GMT
Author: kkolinko
Date: Sun Jan 15 22:14:02 2012
New Revision: 1231773

URL: http://svn.apache.org/viewvc?rev=1231773&view=rev
Log:
Replace description with a formal patch, including the fix from Rainer's review.

Modified:
    tomcat/tc6.0.x/trunk/STATUS.txt

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=1231773&r1=1231772&r2=1231773&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Sun Jan 15 22:14:02 2012
@@ -91,16 +91,12 @@ PATCHES PROPOSED TO BACKPORT:
   -1:
 
 * Reduce log level for the message about hitting maxParameterCount limit
-  from WARN to INFO.
-  in java/org/apache/tomcat/util/http/Parameters.java line 242:
-  - log.warn(sm.getString("parameters.maxCountFail",
-  + log.info(sm.getString("parameters.maxCountFail",
+  from WARN to INFO and fix comparison of parameterCount vs limit that
+  triggers this message.
+  http://people.apache.org/~kkolinko/patches/2012-01-16_tc6_Parameters_maxCountFail.patch
   (backport of http://svn.apache.org/viewvc?rev=1224665&view=rev)
   +1: kkolinko, rjung
   -1:
-  rjung: Why do we compare "parameterCount >= limit" in line 240?
-         Shouldn't it be ">"?
-  kkolinko: I agree, it must be changed to "> limit".
 
 * Update to Apache Commons Pool 1.5.7
   In build.properties.default line 138:



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


Mime
View raw message