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

[Chris Hegarty]

On Tue, 20 Dec 2022 10:27:14 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

>> src/java.net.http/share/classes/jdk/internal/net/http/ResponseSubscribers.java line 284:
>> 
>>> 282:         @Override
>>> 283:         public void onNext(List<ByteBuffer> items) {
>>> 284:             int size = Utils.remaining(items, Integer.MAX_VALUE);
>> 
>> Hello Chris, are we intentionally limiting the `max` to `Integer.MAX_VALUE` here? This specific `Utils.remaining` will throw an `IllegalArgumentException` if the total `remaining` content across the `items` exceed that max value. If it is intentional to use this method, should we explicitly handle the `IAE` and rethrow it as `IOException`?
>
> There's an overloaded `Utils.remaining` which doesn't constrain the maximum size. Perhaps we could use that?

👍 Thanks @jaikiran. Agree. Done.

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

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