https://issues.apache.org/bugzilla/show_bug.cgi?id=55801

--- Comment #5 from Konstantin Kolinko <knst.koli...@gmail.com> ---
It is OK for me, but here are two minor glitches

1. Wasted work.

> SSLContext sslContext = SSLContext.getInstance("TLS");

Such getInstance() calls are usually slow. It'd be better to skip it if you are
ignoring its result.

2. You have Javadoc (good), but these properties are also mentioned at the
bottom of
\webapps\docs\web-socket-howto.xml

It would be good to mention somewhere that the value of a
SSL_TRUSTSTORE_PROPERTY is ignored when using SSL_CONTEXT_PROPERTY.

-- 
You are receiving this mail because:
You are the assignee for the bug.

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

Reply via email to