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

[Evan Whelan]

On Fri, 4 Jun 2021 14:08:58 GMT, Sean Mullan <mullan at openjdk.org> wrote:

>> Evan Whelan has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - Added new line at end of file
>>  - Cleaned up test case
>
> 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 :)

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

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