[8u] TLSv1.3 RFR: 8245653: Remove 8u TLS tests

Martin Balao mbalao at redhat.com
Mon Aug 10 20:52:58 UTC 2020


On 8/10/20 5:19 PM, Alexey Bakhtin wrote:
> OK.
> Webrev renamed to webrev.03 :
> http://cr.openjdk.java.net/~abakhtin/tls1.3/8245466/8245653/webrev.v3/

Thanks for that. Webrev.03 addresses all my comments so far but just
realized there are a few more thing we may need to look at.

In particular, I checked what tests were modified by the original TLS
1.3 patch (8196584) and created a diff list of those not under the
categories covered here so far:

 * test/jdk/ProblemList.txt (modified)
  * do we need to black-list those in JDK-8?
 * test/jdk/com/sun/jndi/ldap/DeadSSLLdapTimeoutTest.java (modified)
  * I suggest including this change.
 * test/jdk/java/net/httpclient/MockServer.java (modified)
  * Not strictly necessary.
 * test/jdk/sun/security/ec/TestEC.java (modified)
  * Do we need this change?
 * test/jdk/sun/security/pkcs11/KeyStore/ClientAuth.java (modified)
  * Do we need this change?
 * test/jdk/sun/security/pkcs11/KeyStore/ClientAuth.sh (modified)
  * Do we need this change?
 * test/jdk/sun/security/tools/keytool/PrintSSL.java (modified)
  * Looks like nice to have.

If you agree, we can handle this in a separated step. Also, in Step 11 I
expect nothing but new files from JDK 11.0.7 without any changes. If you
need modifications on top of these files, please include them on a
separated step -so it's much easier for me to review-.

Let me know what you think.

Thanks,
Martin.-



More information about the jdk8u-dev mailing list