On 17/10/2014 13:18, Konstantin Kolinko wrote: > 2014-10-17 15:15 GMT+04:00 <ma...@apache.org>: >> Author: markt >> Date: Fri Oct 17 11:15:51 2014 >> New Revision: 1632538 >> >> URL: http://svn.apache.org/r1632538 >> Log: >> Proposal >> >> Modified: >> tomcat/tc6.0.x/trunk/STATUS.txt >> > >> +* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=57102 >> + Ensure the the sslEnabledProtocols option works with both BIO and NIO >> + http://people.apache.org/~markt/patches/2014-10-17-bug57102-tc6-v1.patch > > > I think it makes sense to also implement getters for those properties > in Http11NioProtocol. > The NioEndpoint does not have a getter yet, but it has > NioEndpoint.sslEnabledProtocols field that stores original setting.
My aim was the simplest / most obvious patch that fixed the issue. I'm not against further changes but clean-up of this nature is much further down my personal priority list than bug fixes. Mark --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
