Author: markt
Date: Fri Jun 19 21:27:32 2009
New Revision: 786670
URL: http://svn.apache.org/viewvc?rev=786670&view=rev
Log:
Address Konstantin's review comment.
The patch is now sufficiently different that I removed Filip's vote to give him
a chance to re-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=786670&r1=786669&r2=786670&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Fri Jun 19 21:27:32 2009
@@ -198,17 +198,14 @@
+1: kkolinko, markt
-1:
-* Make diagnosing broken requests a little easier
+* Make diagnosing broken requests a little easier. Having to copy the values is
+ a pain but I can't see an alternative. On balance I think it is worth it but
I
+ appreciate others may disagree.
http://svn.apache.org/viewvc?rev=783724&view=rev
http://svn.apache.org/viewvc?rev=786468&view=rev
- +1: markt, fhanik
- -1: kkolinko (
- Regarding the second part of the patch:
- queryMB is not always the "original data" for the call.
- o.a.c.connector.Request#parseParameters() calls processParameters() as
well,
- to parse body of a POST request. Though it will be a rare case to
trigger
- this log message from that call.
- )
+ http://svn.apache.org/viewvc?rev=786667&view=rev
+ +1: markt
+ -1:
* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=47343
Regression in fix for
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
