On Thu, 30 Apr 2026 17:26:14 GMT, Weijun Wang <[email protected]> wrote:
>> Artur Barashev 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 five additional
>> commits since the last revision:
>>
>> - Add @throws section to getUseClientMode()
>> - Merge branch 'master' into JDK-8382270
>> - Throw an exception when getUseClientMode() is called before
>> setUseClientMode(). Remove any notion of default from SSLEngine javadoc
>> - Address review comments
>> - 8382270: Update SSLContex API javadoc: require setUseClientMode call for
>> SunJSSE
>
> src/java.base/share/classes/javax/net/ssl/SSLContext.java line 352:
>
>> 350: * Some cipher suites (such as Kerberos) require remote hostname
>> 351: * information, in which case this factory method should not be
>> used.
>> 352: * <P>
>
> Put `SSLEngine` here and the one at the end of the `@implNote` into `{@code}`.
Done, thanks!
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/30754#discussion_r3170101628
