Author: markt
Date: Wed Apr 7 21:54:17 2010
New Revision: 931701
URL: http://svn.apache.org/viewvc?rev=931701&view=rev
Log:
Vote
Update patch to address comments
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=931701&r1=931700&r2=931701&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Wed Apr 7 21:54:17 2010
@@ -194,17 +194,15 @@ PATCHES PROPOSED TO BACKPORT:
Mark ResourceEntry.loadedClass as volatile.
http://svn.apache.org/viewvc?rev=927877&view=rev
- +1: kkolinko
+ +1: kkolinko, markt
-1:
* Correct SSL session timeout attribute name
- http://people.apache.org/~markt/patches/2010-03-18-SslSessionTimeout.patch
+ http://people.apache.org/~markt/patches/2010-04-07-SslSessionTimeout.patch
+1: markt
-1:
- kkolinko: In NioEndpoint (and in AbstractEndpoint in trunk) there is
- #setSessionCacheTimeout(String). It will need renaming as well?
- Generally, I would be against renaming some attributes, but this one
- is already documented as "sessionTimeout" in config/http.xml, though,
+ kkolinko: Generally, I would be against renaming some attributes, but this
+ one is already documented as "sessionTimeout" in config/http.xml, though,
still, renaming might break someone's configurations.
* Fix cluster regression, previous incorrect patch
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]