[8u] TLSv1.3 RFR: 8245681: Add TLSv1.3 regression test from 11.0.7

Alexey Bakhtin alexey at azul.com
Wed Jul 22 18:44:08 UTC 2020


Please find updated patch for TLSv1,3 regression tests:
http://cr.openjdk.java.net/~abakhtin/tls1.3/8245466/8245681/webrev.v1

This version fixes test issues caused by client default disabled TLSv1.3 protocol :
- javax/net/ssl/SSLSession/ResumeTLS13withSNI.java
- javax/net/ssl/SSLSocket/Tls13PacketSize.java
- javax/net/ssl/Stapling/HttpsUrlConnClient.java
- javax/net/ssl/Stapling/SSLEngineWithStapling.java
- javax/net/ssl/Stapling/SSLSocketWithStapling.java
- javax/net/ssl/Stapling/StapleEnableProps.java
- javax/net/ssl/sanity/ciphersuites/CheckCipherSuites.java

Also, this version reverts incorrectly modified  sun/security/krb5/auto/SSL.java test

Regards
Alexey

> On 28 May 2020, at 19:56, Alexey Bakhtin <alexey at azul.com> wrote:
> 
> Hello Dmitry,
> 
> Yes, you are right.
> I’ve uploaded non-incremental webrev for the JDK-8245653 "Remove 8u TLS tests” by mistake.
> I’ve updated it in the same location : http://cr.openjdk.java.net/~abakhtin/tls1.3/8245466/8245653/webrev.v0/
> 
> JDK-8245681 should be applied cleanly now
> 
> Thank you
> Alexey
> 
> 
>> On 28 May 2020, at 19:07, Dmitry Cherepanov <dcherepanov at azul.com> wrote:
>> 
>> Hi Alexey,
>> 
>> On 24.05.2020 21:05, Alexey Bakhtin wrote:
>>> Please review changes required to backport TLSv1.3 protocol from JDK11.0.7 to JDK8u
>>> 
>>> JBS task: https://bugs.openjdk.java.net/browse/JDK-8245681
>>> Webrev: http://cr.openjdk.java.net/~abakhtin/tls1.3/8245466/8245681/webrev.v0/
>>> 
>>> This is a third part of TLS test update.
>> 
>> When I tried to apply patches, I got conflicts in the last step. There are a lot of rejects due to the fact that new test files already exist.
>> 
>> For example, the third part (of test update) adds new file - test/javax/net/ssl/ALPN/SSLEngineAlpnTest.java
>> which existed before and was moved by the first part (of test update)  from test/sun/security/ssl/javax/net/ssl/ALPN/SSLEngineAlpnTest.java to test/javax/net/ssl/ALPN/SSLEngineAlpnTest.java
>> 
>> Could you please double check that it's the latest version of webrev?
>> 
>> Thanks
>> 
>> Dmitry
>> 



More information about the jdk8u-dev mailing list