Re: RFR: JDK-8207237: SSLSocket#setEnabledCipherSuites is accepting empty string

2018-07-17 Thread Xuelei Fan
Looks fine to me. Thanks, Xuelei > On Jul 17, 2018, at 9:43 PM, Jamil Nimeh wrote: > > Hello all, > > This fixes a recent finding in the new TLS code where we're accepting a > cipher suite that is an empty string where in the past we would throw > IllegalArgumentException. This fix makes th

RFR: JDK-8207237: SSLSocket#setEnabledCipherSuites is accepting empty string

2018-07-17 Thread Jamil Nimeh
Hello all, This fixes a recent finding in the new TLS code where we're accepting a cipher suite that is an empty string where in the past we would throw IllegalArgumentException.  This fix makes the new TLS implementation consistent with the old behavior. Webrev: http://cr.openjdk.java.net/~