Author: markt
Date: Mon Nov  2 21:55:55 2009
New Revision: 832122

URL: http://svn.apache.org/viewvc?rev=832122&view=rev
Log:
Withdraw my patch - vote for Konstantin's

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=832122&r1=832121&r2=832122&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Mon Nov  2 21:55:55 2009
@@ -83,20 +83,12 @@
   -1:
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=39231
-  http://people.apache.org/~markt/patches/2009-08-16-bug39231.patch
   JAAS LoginContext expects a call to logout()
-  +1: markt, jim, fhanik
-  -1: kkolinko: It works for classes that call 
JAASRealm#createPrincipal(String,Subject),
-    but breaks those ones that overwrite that method, because overwritten
-    method will not be called by the base class. An alternative patch is 
proposed below.
-
-* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=39231
-  It is updated version of Mark's patch,
+  It is updated version of Mark's (since withdrawn) patch,
   where the new method in JAASRealm calls the old one.
   http://people.apache.org/~kkolinko/patches/2009-11-02_bug39231.patch
-  +1: kkolinko
-  -1:
-
+  +1: kkolinko, markt
+  -1: 
  
 * Improve NIO connector shutdown time by doing shutdowns in parallel
   and with a timeout



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to