RFR: 8299015: Ensure that HttpResponse.BodySubscribers.ofFile writes all bytes [v5]

Daniel Fuchs dfuchs at openjdk.org
Tue Dec 20 14:54:54 UTC 2022


On Tue, 20 Dec 2022 14:44:02 GMT, Chris Hegarty <chegar at openjdk.org> wrote:

>> src/java.net.http/share/classes/jdk/internal/net/http/ResponseSubscribers.java line 305:
>> 
>>> 303:                     }
>>> 304:                 }
>>> 305:             } catch (Exception ex) {
>> 
>> I believe we should change that line to `catch (IOException | ArithmeicException ex) {`
>
> Agreed. Testing it locally before pushing.

Alternatively it seems that moving the try down to line 292 works too.

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

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


More information about the net-dev mailing list