RFR: 8044609: javax.net.debug options not working and documented as expected [v18]

Sean Coffey coffeys at openjdk.org
Mon Sep 1 15:31:56 UTC 2025


On Sat, 30 Aug 2025 01:16:01 GMT, Bradford Wetmore <wetmore at openjdk.org> wrote:

>> Sean Coffey has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 35 commits:
>> 
>>  - Merge branch 'master' into 8044609-ssl
>>  - 1 file omitted during merge
>>  - Merge branch 'master' into 8044609-ssl
>>  - Merge branch 'master' into 8044609-ssl
>>  - Merge branch 'master' into 8044609-ssl
>>  - remove whitespace
>>  - erroneous edit to test file
>>  - remove legacy test and minor fix ups
>>  - Merge branch 'master' into 8044609-ssl
>>  - enums refactoring and line width correction
>>  - ... and 25 more: https://git.openjdk.org/jdk/compare/2b44ed70...71aa0211
>
> src/java.base/share/classes/sun/security/ssl/SSLLogger.java line 203:
> 
>> 201:         System.err.println("\thandshake    print each handshake message");
>> 202:         System.err.println("\tkeymanager   print key manager tracing");
>> 203:         System.err.println("\trecord       enable per-record tracing");
> 
> What would you think of listing/indenting the suboptions here instead of down below?  e.g. 
> 
>     record             enable per-record tracing
>         plaintext         hex dump of record plaintext  (widens record)
>         packet            print raw SSL/TLS packets (widens record)
> 
> same with `verbose`

Good idea. I've gone with that approach.

> src/java.base/share/classes/sun/security/ssl/SSLLogger.java line 223:
> 
>> 221:     }
>> 222: 
>> 223:     public enum Opt {
> 
> A quick comment about the enum's purpose might be appreciated by someone new.  "list of options, whether they are active..."

Added some text to indicate purpose of the new enum type

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18764#discussion_r2314229266
PR Review Comment: https://git.openjdk.org/jdk/pull/18764#discussion_r2314230004


More information about the security-dev mailing list