Author: kkolinko
Date: Wed Jul 15 00:08:56 2009
New Revision: 794119

URL: http://svn.apache.org/viewvc?rev=794119&view=rev
Log:
vote

Modified:
    tomcat/current/tc5.5.x/STATUS.txt

Modified: tomcat/current/tc5.5.x/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/current/tc5.5.x/STATUS.txt?rev=794119&r1=794118&r2=794119&view=diff
==============================================================================
--- tomcat/current/tc5.5.x/STATUS.txt (original)
+++ tomcat/current/tc5.5.x/STATUS.txt Wed Jul 15 00:08:56 2009
@@ -56,8 +56,12 @@
   AJP connectors do not handle certificate chains
   Patch by Patrik Schnellmann
   https://issues.apache.org/bugzilla/attachment.cgi?id=23952
-  +1: markt
+  +1: markt, kkolinko
   -1: 
+  kkolinko: in the lines of RequestHandler.java that are seen in the patch,
+    but are not changed by it there is a bug:  "certString.getBytes()" - it
+    uses Platform default encoding. Though it should not be of much value
+    for base64 encoded data that these certificates are.
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=39844
   Port r588477 (fix for #43668) by billbarker that corrected this for Tomcat 6



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

Reply via email to