On Tue, 25 Nov 2025 10:46:34 GMT, Daniel Jeliński <[email protected]> wrote:
>> When checking for supported TLS versions, use SSLContext default parameters >> if the provided SSLParameters have no protocols configured. >> >> This fixes an issue where using SSLParameters with no protocols disabled the >> use of HTTP2 and HTTP3, even when these protocols were supported and enabled >> in the SSLContext. >> >> Modified the existing tests to additionally cover the case of empty >> SSLParameters. All tests continue to pass. > > Daniel Jeliński has updated the pull request incrementally with two > additional commits since the last revision: > > - Cleanup HttpClientImpl ctor > - Rename setParams These changes look good to me. ------------- Marked as reviewed by jpai (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/28480#pullrequestreview-3505004311
