You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2010/11/25 18:40:06 UTC

svn commit: r1039109 - /tomcat/tc6.0.x/trunk/STATUS.txt

Author: markt
Date: Thu Nov 25 17:40:05 2010
New Revision: 1039109

URL: http://svn.apache.org/viewvc?rev=1039109&view=rev
Log:
Vote

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=1039109&r1=1039108&r2=1039109&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Thu Nov 25 17:40:05 2010
@@ -101,20 +101,8 @@ PATCHES PROPOSED TO BACKPORT:
 
 * Fix path parameter handling. Currently the following URL fails with a 404:
   http://localhost:8080/examples/jsp/snp;x=y/snoop.jsp
-  http://people.apache.org/~markt/patches/2010-10-06-path-param-tc6.patch
-  +1: markt
-  +1: kkolinko:
-    with the following trivial changes:
-     - with r1022389 and r1035973
-     - with access logging as done by r1030188
-     - Mark CoyoteAdapter.parseSessionId(..) as @deprecated, because it is no more used
-       In the updated patch below I replaced its implementation with a call to parsePathParameters().
-     - Add 'coyoteAdapter.parsePathParam' message to org/apache/catalina/connector/LocalStrings.properties
-  -1:
-
-    Updated patch, based on markt's one and my above comments:
   http://people.apache.org/~kkolinko/patches/2010-11-17_tc6_path-params.patch
-  +1: kkolinko
+  +1: kkolinko, markt
   -1:
 
    kkolinko: The old parseSessionId() method calls "clearRequestedSessionURL(request);"
@@ -195,6 +183,9 @@ PATCHES PROPOSED TO BACKPORT:
   Combined TC 6 patch:
   http://people.apache.org/~rjung/patches/BZ48358-JSP_Unloading-TC6_20101118.patch
   +1: rjung
+  -0: markt - http://markmail.org/thread/777be426ulcfmdd3 suggests there may be
+              a memory leak in this code somewhere. I'd like to get to the
+              bottom of that before porting this
   -1:
 
 * Add session creation / expiration rate statistics to the session managers
@@ -207,23 +198,24 @@ PATCHES PROPOSED TO BACKPORT:
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=50228
   Improve recycling of BodyContentImpl.
   http://svn.apache.org/viewvc?rev=1037794&view=rev
-  +1: kkolinko
+  +1: kkolinko, markt
   -1:
 
 * Configure manager and host-manager webapps to use HttpOnly for session cookies
   http://people.apache.org/~kkolinko/patches/2010-11-22_tc6_httpOnly.patch
-  +1: kkolinko
+  +1: kkolinko, markt
   -1:
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=50316
   Fix minor glitch with display of negative values in Manager webapp
   http://svn.apache.org/viewvc?rev=1037887&view=rev
-  +1: kkolinko
+  +1: kkolinko, markt
   -1:
 
 * Backport AprEndpoint shutdown patch (BZ49795 and similar).
   http://people.apache.org/~mturk/tomcat/patches/tomcat-6.0.x-aprshutdown.patch
   +1: mturk
+  -0: markt - Patch doesn't apply cleanly to tc6.0.x/trunk
   -1:
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48837



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


Re: svn commit: r1039109 - /tomcat/tc6.0.x/trunk/STATUS.txt

Posted by Mladen Turk <mt...@apache.org>.
On 11/25/2010 06:40 PM, markt@apache.org wrote:
> Author: markt
>
>   * Backport AprEndpoint shutdown patch (BZ49795 and similar).
>     http://people.apache.org/~mturk/tomcat/patches/tomcat-6.0.x-aprshutdown.patch
>     +1: mturk
> +  -0: markt - Patch doesn't apply cleanly to tc6.0.x/trunk


Sure since it was written and proposed before your r1039103 commit.


Regards
-- 
^TM

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