tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ma...@apache.org
Subject svn commit: r892760 - /tomcat/tc6.0.x/trunk/STATUS.txt
Date Mon, 21 Dec 2009 10:54:14 GMT
Author: markt
Date: Mon Dec 21 10:54:14 2009
New Revision: 892760

URL: http://svn.apache.org/viewvc?rev=892760&view=rev
Log:
Simplify proposal. No actual changes, just putting everything in a single file so leaving
Konstantin's vote in place.

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=892760&r1=892759&r2=892760&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Mon Dec 21 10:54:14 2009
@@ -357,31 +357,6 @@
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=47963
   Prevent use of non-RFC2616 compliant custom status messages
-  http://svn.apache.org/viewvc?rev=892612&view=rev
-  http://svn.apache.org/viewvc?rev=892707&view=rev
-  +1: markt
-  +1: kkolinko: (
-       1. only together with rev.892707
-       2. this change, provided by rev.892612,
-
--        if (org.apache.coyote.Constants.USE_CUSTOM_STATUS_MSG_IN_HEADER) {
-+        if (org.apache.coyote.Constants.USE_CUSTOM_STATUS_MSG_IN_HEADER &&
-+                HttpMessages.isSafeInHttpHeader(response.getMessage())) {
-             message = response.getMessage();
-         }
-         if (message == null){
-             message = HttpMessages.getMessage(response.getStatus());
--        } else {
--            message = message.replace('\n', ' ').replace('\r', ' ');
-         }
-
-        has to be applied to the following 6 classes in TC 6:
-         in o.a.coyote.ajp: AjpAprProcessor and AjpProcessor,
-         in o.a.coyote.http11: InternalAprOutputBuffer, InternalNioOutputBuffer, InternalOutputBuffer
-         in o.a.jk.common: JkInputStream
-        (3 of them do not exist in TC 7).
-
-       3. I think that mention of extra overhead in systemprops.xml is not
-       necessary.
-     )
+  http://people.apache.org/~markt/patches/2009-12-21-bug47963.patch
+  +1: markt, kkolinko
   -1: 



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org


Mime
View raw message