RFR: 8376031: HttpsURLConnection.getServerCertificates() throws "java.lang.IllegalStateException: connection not yet open" for the HEAD method [v3]
Daniel Fuchs
dfuchs at openjdk.org
Tue Feb 3 18:29:24 UTC 2026
On Sun, 1 Feb 2026 11:56:49 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:
>> Daniel Fuchs has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:
>>
>> - Review feedback: improve the test to verify that certificates are returned
>> - Merge branch 'master' into https_certs_8376031
>> - Review feedback
>> - 8376031: HttpsURLConnection.getServerCertificates() throws "java.lang.IllegalStateException: connection not yet open" for the HEAD method
>
> test/jdk/sun/net/www/protocol/https/HttpsURLConnection/GetServerCertificates.java line 151:
>
>> 149: .formatted(code, resp));
>> 150:
>> 151: uc.getServerCertificates();
>
> Here and a few other places in this test, should we assert that this returns non-null certificates?
@jaikiran I have added a check to verify that the returned certs are not null
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29489#discussion_r2760412654
More information about the net-dev
mailing list