On Wed, 22 Jan 2025 09:45:16 GMT, Volkan Yazici <vyaz...@openjdk.org> wrote:

>> Adds `limiting()` factory methods to 
>> `HttpResponse.Body{Handlers,Subscribers}` to handle excessive server input 
>> in `HttpClient`. I would appreciate your input whether `discardExcess` 
>> should be kept or dropped. I plan to file a CSR once there is an agreement 
>> on the PR.
>
> Volkan Yazici has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Verify `limiting()` doesn't affect header parsing

Overall, these changes look good to me. There are just a few things to address 
in the test, which I have added inline.

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

PR Comment: https://git.openjdk.org/jdk/pull/23096#issuecomment-2607504685

Reply via email to