On Wed, 15 Apr 2026 19:14:55 GMT, Artur Barashev <[email protected]> wrote:

>> `InterruptedIOException` is being deprecated, replace it with 
>> `SocketTimeoutException` in SSL code.
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Artur Barashev has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Update Copyright year

src/java.base/share/classes/sun/security/ssl/SSLSocketImpl.java line 1682:

> 1680:         // Don't close the Socket in case of timeouts or interrupts.
> 1681:         if (cause instanceof SocketTimeoutException) {
> 1682:             throw (IOException)cause;

You could use pattern matching for switch here and avoid the cast to 
IOException.

test/jdk/sun/security/ssl/SSLSocketImpl/ClientTimeout.java line 29:

> 27: /*
> 28:  * @test
> 29:  * @bug 4836493 8239798 8370656

I don't think you need to add to this `@bug` tag.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30753#discussion_r3091257061
PR Review Comment: https://git.openjdk.org/jdk/pull/30753#discussion_r3091260426

Reply via email to