On Thu, 30 Oct 2025 10:16:16 GMT, Mikhail Yankelevich 
<[email protected]> wrote:

>> Updated test to require `ssl,keymanager`, as it changed from `keymanager`.
>
> Mikhail Yankelevich has updated the pull request incrementally with two 
> additional commits since the last revision:
> 
>  - turbanoff's comment
>  - choosing -> Choosing

Took way to long to figure out that a capital letter changed in the SSL debug 
output, and that was causing the problem.   :)

Thank you for cleaning up the formatting.  Having lines <=80 sure helps with 
side/side comparisons like here in github.  

LGTM.

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

Marked as reviewed by wetmore (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/28045#pullrequestreview-3401109018

Reply via email to