RFR: JDK-8299475: Enhance SocketException by cause where it is missing in net and nio area [v2]

Matthias Baesken mbaesken at openjdk.org
Tue Jan 3 08:29:14 UTC 2023


On Mon, 2 Jan 2023 16:34:26 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> Matthias Baesken has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   do not enhance some internal exceptions
>
> src/java.base/share/classes/java/net/ServerSocket.java line 697:
> 
>> 695:             if (thread.isVirtual() && thread.isInterrupted()) {
>> 696:                 close();
>> 697:                 throw new SocketException("Closed by interrupt", e);
> 
> The InterruptedIOException is used by the SocketImpl to communicate the interrupt to the ServerSocket. It should not be added a cause here. Same thing in the Socket code.

Hi Alan, I removed this one.

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

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


More information about the nio-dev mailing list