[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