On Thu, 16 Feb 2023 18:40:56 GMT, Rajan Halade <rhal...@openjdk.org> wrote:
>> * Refactored SSLContextTemplate and SSLSocketTemplate to put common code in
>> one base class (SSLContextTemplate)
>> * Updated TLS/SSL tests to extend SSLSocketTemplate where possible.
>> * Updated SSLEngineTemplate to accommodate changes in SSLContextTemplate. To
>> keep this changeset to a reasonable size, updates to SSLEngine tests will be
>> made under JDK-8301194.
>
> test/jdk/javax/net/ssl/templates/SSLContextTemplate.java line 395:
>
>> 393: CA_RSA_512( // for DisabledShortRSAKeys test
>> 394: "RSA",
>> 395: "-----BEGIN CERTIFICATE-----\n" +
>
> Please add a comment with validity and algorithms similar to other entries in
> this enum. There are few other entries to update as well.
done
> test/jdk/sun/net/www/protocol/https/HttpsClient/ServerIdentityTest.java line
> 111:
>
>> 109:
>> 110: @Override
>> 111: public SSLContext createServerSSLContext() throws Exception {
>
> revert this as well.
reverted
-------------
PR: https://git.openjdk.org/jdk/pull/12555
