Looks fine to me.
Xuelei
On 7/25/2018 8:14 AM, Jamil Nimeh wrote:
Hi folks,
This is a review for a backport to the JDK 11 repo of a change I did
earlier to jdk/jdk that fixed the SSLSocket.setEnabledCipherSuites
method. It no longer accepts empty strings in the cipher suite list.
Instread, it returns to the behavior of throwing
IllegalArgumentException as it did before the new TLS handshaker code
was committed.
Webrev: http://cr.openjdk.java.net/~jnimeh/reviews/8207237-jdk11/webrev.01/
JBS: https://bugs.openjdk.java.net/browse/JDK-8207237
Thanks,
--Jamil
