Yep, code review coming in a few minutes.
--Jamil
On 9/21/2018 2:22 PM, Bradford Wetmore wrote:
Looks fine.
Where are your new regression tests for 916/334? Will that be under a
separate bug?
Thanks.
Brad
On 9/20/2018 11:51 PM, Jamil Nimeh wrote:
Hello all,
This review is for a backport of 3 TLS interoperability issues that
have come up over the past week or so. These are already in
jdk/jdk. They cover the following issues:
* An NPE thrown during processing of the supported groups extension
with curves not enabled by default
* A problem processing client hellos where pre_shared_key and
psk_key_exchange_modes extensions are both omitted
* A problem where JSSE TLS clients send empty Certificate messages
during client certificate authentication
Webrev:
http://cr.openjdk.java.net/~jnimeh/reviews/jdk11u-tls-compat/webrev.01/
JBS Issues:
https://bugs.openjdk.java.net/browse/JDK-8209916
https://bugs.openjdk.java.net/browse/JDK-8210334
https://bugs.openjdk.java.net/browse/JDK-8210846
Thanks,
--Jamil
