In what scenario would the SSL filter be attached but the session has no filterchain
On Thu, Jan 28, 2016 at 7:40 AM, Emmanuel Lécharny <elecha...@gmail.com> wrote: > Le 28/01/16 13:33, Norbert Irmer a écrit : > > Your idea, adding a method "public void initiateHandshake(IoSession > session)" to the SslFilter class, > > would be perfectly fine for me. > > > > I just tested it, and it works fine. Only thing, perhaps, is, that I > don't know, if session.getFilterChain > > can return null, so perhaps some null pointer handling is necessary. > > > > > Yes, actually, I have a better implementation that tests corner cases : > > public void initiateHandshake(IoSession session) throws SSLException { > IoFilterChain filterChain = session.getFilterChain(); > > if (filterChain == null) { > throw new SSLException("No filter chain"); > } > > IoFilter.NextFilter nextFilter = > filterChain.getNextFilter(SslFilter.class); > > if (nextFilter == null) { > throw new SSLException("No SSL next filter in the chain"); > } > > initiateHandshake(nextFilter, session); > } > > > This is what I'm going to commit. >