tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
Subject svn commit: r658132 - /tomcat/tc6.0.x/trunk/STATUS.txt
Date Tue, 20 May 2008 07:34:01 GMT
Author: markt
Date: Tue May 20 00:34:01 2008
New Revision: 658132

Add some comments to Remy's -1s


Modified: tomcat/tc6.0.x/trunk/STATUS.txt
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Tue May 20 00:34:01 2008
@@ -68,6 +68,8 @@
   +1: markt
   -1: remm (I think it is a good patch, but it throws an error for something which was working
             so I would think keeping it for the next release would be better)
+      markt How about making it optional with a system property that can be used
+            to restore the current behaviour is an app depends on it?
 * Fix
   Correct BNF grammar so ${0 lt a ? 1 lt a ? "many": "one": "none"} works
@@ -93,6 +95,10 @@
             it may not apply to HTTP/1.1, which may be more strict; passing full URLs in
             request line is normally an HTTP/1.1 thing, from what I remember, usable as an
             to the mandatory Host header)
+      markt Both the HTTP/1.1 and HTTP/1.0 contain broadly the same text
+            regarding tolerant applications. (ie allow multiple SP/HT between
+            method-url and url-httpVersion). However, it is optional so we don't
+            have to support it.
 * Fix
   Use any META-INF/context.xml in a consistent manner and ensure it is used on
@@ -100,3 +106,6 @@
   +1: markt
   -1: remm (would need convincing that this is really well tested, otherwise delay to next
+      markt Exactly what would convince you that this was well tested above and
+            beyond the local testing that I have done to convince myself that
+            this patch does what I want it to.
\ No newline at end of file

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message