You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by kk...@apache.org on 2012/01/15 23:14:02 UTC
svn commit: r1231773 - /tomcat/tc6.0.x/trunk/STATUS.txt
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