Author: kkolinko
Date: Fri Dec 18 03:43:42 2009
New Revision: 892120
URL: http://svn.apache.org/viewvc?rev=892120&view=rev
Log:
votes
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=892120&r1=892119&r2=892120&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Fri Dec 18 03:43:42 2009
@@ -362,7 +362,7 @@
* Prevent lost log messages on shutdown
http://svn.apache.org/viewvc?rev=888072&view=rev
- +1: markt, jim
+ +1: markt, jim, kkolinko
-1:
* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=47537
@@ -405,6 +405,21 @@
http://svn.apache.org/viewvc?rev=890530&view=rev
+1: markt
-1:
+ 0: kkolinko:
+ Re approach:
+ Request.isRequestedSessionIdValid() serves as implementation for
+ HttpServletRequest.isRequestedSessionIdValid().
+ I think that ClassLoader should already be set when calling this
+ method.
+
+ I mean, the problem is elsewhere. Supposedly in CoyoteAdapter.
+ parseSessionCookiesId(), but may be earlier in the call chain.
+
+ I won't oppose the patch. I have to think a bit more about it.
+
+ Re implementation:
+ if (session == null) return false; // return early and do not bother
with thread class loader
+
* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=47836
Don't keep TLD/listener info between reloads
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
