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-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org