* Reduce log level for the message about hitting maxParameterCount limit
from WARN to INFO.
git-svn-id: https://svn.apache.org/repos/asf/tomcat/tc5.5.x/trunk@1228185 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/STATUS.txt b/STATUS.txt
index 67d29de..c73b3e2 100644
--- a/STATUS.txt
+++ b/STATUS.txt
@@ -24,18 +24,6 @@
PATCHES ACCEPTED TO BACKPORT FROM TRUNK/OTHER:
[ start all new proposals below, under PATCHES PROPOSED. ]
-* Reduce log level for the message about hitting maxParameterCount limit
- from WARN to INFO.
- in /connectors/util/java/org/apache/tomcat/util/http/Parameters.java line 243:
- - log.warn(sm.getString("parameters.maxCountFail",
- + log.info(sm.getString("parameters.maxCountFail",
- (backport of http://svn.apache.org/viewvc?rev=1224665&view=rev)
- +1: kkolinko, rjung, jim (w/ ">" change)
- -1:
- rjung: Why do we compare "parameterCount >= limit" in line 241?
- Shouldn't it be ">"?
- kkolinko: I agree, it must be changed to "> limit".
-
PATCHES PROPOSED TO BACKPORT:
[ New proposals should be added at the end of the list ]
diff --git a/connectors/util/java/org/apache/tomcat/util/http/Parameters.java b/connectors/util/java/org/apache/tomcat/util/http/Parameters.java
index 2c40fb9..79a1a17 100644
--- a/connectors/util/java/org/apache/tomcat/util/http/Parameters.java
+++ b/connectors/util/java/org/apache/tomcat/util/http/Parameters.java
@@ -238,9 +238,9 @@
while(pos < end) {
parameterCount ++;
- if (limit > -1 && parameterCount >= limit) {
+ if (limit > -1 && parameterCount > limit) {
parseFailed = true;
- log.warn(sm.getString("parameters.maxCountFail",
+ log.info(sm.getString("parameters.maxCountFail",
Integer.toString(limit)));
break;
}
