RFR: 8299827: Add resolved IP address in connection exception for sockets [v4]

Ralf Schmelter rschmelter at openjdk.org
Wed Jan 18 09:42:39 UTC 2023


On Wed, 18 Jan 2023 06:50:25 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

>> Ralf Schmelter has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Accidentally changed Unix domain socket exception text instead of socket exception text
>
> test/jdk/java/net/Socket/ExceptionText.java line 114:
> 
>> 112:                     return; // has no detail msg
>> 113:                 System.err.println("msg = " + msg);
>> 114:                 throw new RuntimeException("Test failed: exception does not contain address info: " + msg);
> 
> It might help debugging any unexpected failures, if the exception message here could include the expected address. Something like:
> 
> throw new RuntimeException("Test failed: exception message " + msg + " does not contain address " + dest);

I've added the destination and removed the System.err.println() call above, since it now seems redundant.

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

PR: https://git.openjdk.org/jdk/pull/11961


More information about the net-dev mailing list