RFR: JDK-8260925: HttpsURLConnection does not work with other JSSE provider. [v4]

[Vyom Tewari]

On Wed, 3 Mar 2021 16:16:48 GMT, Xue-Lei Andrew Fan <xuelei at openjdk.org> wrote:

>> Vyom Tewari has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   reverted the overly deleted else block
>
> src/java.base/share/classes/sun/net/www/protocol/https/HttpsClient.java line 566:
> 
>> 564:                     SSLParameters paramaters = s.getSSLParameters();
>> 565:                     paramaters.setEndpointIdentificationAlgorithm("HTTPS");
>> 566:                     paramaters.setServerNames(List.of(new SNIHostName(host)));
> 
> Thank you for taking my comment.  But I may not update line 456-458.  A safer update may be in line 566, for example:
> +    // The host has been set for SSLSocketImp previously.
> +    if (!(s instanced SSLSocketImpl)) {
>           paramaters.setServerNames(List.of(new SNIHostName(host)));
> +    }

I did the changes as you suggested.

-------------

PR: https://git.openjdk.java.net/jdk/pull/2583