RFR: 8255148: Confusing log output: SSLSocket duplex close failed [v2]

[Sean Mullan]

On Wed, 9 Jun 2021 17:43:45 GMT, Evan Whelan <ewhelan at openjdk.org> wrote:

>> src/java.base/share/classes/sun/security/ssl/SSLSocketImpl.java line 590:
>> 
>>> 588:             // ignore the exception
>>> 589:             if (SSLLogger.isOn && SSLLogger.isOn("ssl")) {
>>> 590:                 SSLLogger.warning("SSLSocket duplex close failed. Debug info only. Exception details:", ioe);
>> 
>> If this is a debug message, shouldn't we just use `SSLLogger.fine()` instead of `SSLLogger.warning()`, with the same message "SSLSocket duplex close failed"? @coffeys what do you think?
>
> Hi @seanjmullan @coffeys 
> 
> I also don't have a logging level preference, so if there's merit to changing it, I'll be happy to do so.
> I've updated the test case and verified it passes on all platforms.
> 
> Looking forward to any further feedback :)

Ok, It sounds like whether it should be a different logging level (fine) or not can be handled separately as part of a more global effort across other SSL logging messages. So I am ok with the change as-is.

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

PR: https://git.openjdk.java.net/jdk/pull/4354