[11u] RFR: 8249159: Downport test rework for SSLSocketTemplate from 8224650
Baesken, Matthias
matthias.baesken at sap.com
Fri Jul 10 11:08:55 UTC 2020
I would like to bring this test rework from 14 to 11.
It is part of https://bugs.openjdk.java.net/browse/JDK-8224650 .
8224650 contains the rework of SSLSocketTemplates.java and
adds a new test.
In the old version of SSLSocketTemplates.java certificates, names and keys
are held in separate arrays that must correlate by indexes correctly. The new
version uses enums to keep common information in one place. The new
version is much easier to maintain. Also, downports that just add new enums
are easier to handle.
The new test NamedGroupsWithCipherSuite.java does not pass in 11 because
the feature tested (JDK-8171279) is only in 13.
I would like to downport the rework because it makes JDK-8246330 apply
clean. I do not want to downport the new test as it fails. Therefore I opened
an Enhancement issue of it's own for this, https://bugs.openjdk.java.net/browse/JDK-8249159 .
Please review this webrev:
http://cr.openjdk.java.net/~goetz/wr20/8249159-rework_SSLSocketTemplate-jdk11/01/
It contains the changes of SSLSocketTemplate from JDK-8224650. They applied clean.
It does not contain the new test.
SSLSocketTemplate.java passes with this test.
Best regards,
Goetz.
---------------------------------------------------------
Hi Götz , while the patch downport looks mostly fine, I wonder about one point.
It looks to me like the certs tested changed a bit with your change . If this is really intentional , please confirm and consider your change as reviewed .
Otherwise I wonder about for example this cert, I only find it in the new version ( did not check all the certs but noticed this one ).
Best regards, Matthias
http://cr.openjdk.java.net/~goetz/wr20/8249159-rework_SSLSocketTemplate-jdk11/01/test/jdk/javax/net/ssl/templates/SSLSocketTemplate.java.frames.html
636 CA_ECDSA_SECP384R1(
637 "EC",
638 // SHA384withECDSA, curve secp384r1
639 // Validity
640 // Not Before: Jun 24 08:15:06 2019 GMT
641 // Not After : Jun 19 08:15:06 2039 GMT
642 // Subject Key Identifier:
643 // 0a:93:a9:a0:bf:e7:d5:48:9d:4f:89:15:c6:51:98:80:05:51:4e:4e
644 "-----BEGIN CERTIFICATE-----\n" +
645 "MIICCDCCAY6gAwIBAgIUCpOpoL/n1UidT4kVxlGYgAVRTk4wCgYIKoZIzj0EAwMw\n" +
646 "OzELMAkGA1UEBhMCVVMxDTALBgNVBAoMBEphdmExHTAbBgNVBAsMFFN1bkpTU0Ug\n" +
647 "VGVzdCBTZXJpdmNlMB4XDTE5MDYyNDA4MTUwNloXDTM5MDYxOTA4MTUwNlowOzEL\n" +
648 "MAkGA1UEBhMCVVMxDTALBgNVBAoMBEphdmExHTAbBgNVBAsMFFN1bkpTU0UgVGVz\n" +
649 "dCBTZXJpdmNlMHYwEAYHKoZIzj0CAQYFK4EEACIDYgAENVQN1wXWFdgC6u/dDdiC\n" +
650 "y+WtMTF66oL/0BSm+1ZqsogamzCryawOcHgiuXgWzx5CQ3LuOC+tDFyXpGfHuCvb\n" +
651 "dkzxPrP5n9NrR8/uRPe5l1KOUbchviU8z9cTP+LZxnZDo1MwUTAdBgNVHQ4EFgQU\n" +
652 "SktSFArR1p/5mXV0kyo0RxIVa/UwHwYDVR0jBBgwFoAUSktSFArR1p/5mXV0kyo0\n" +
653 "RxIVa/UwDwYDVR0TAQH/BAUwAwEB/zAKBggqhkjOPQQDAwNoADBlAjBZvoNmq3/v\n" +
654 "RD2gBTyvxjS9h0rsMRLHDnvul/KWngytwGPTOBo0Y8ixQXSjdKoc3rkCMQDkiNgx\n" +
655 "IDxuHedmrLQKIPnVcthTmwv7//jHiqGoKofwChMo2a1P+DQdhszmeHD/ARQ=\n" +
656 "-----END CERTIFICATE-----",
More information about the jdk-updates-dev
mailing list