On Mon, 20 Jan 2025 08:31:18 GMT, Volkan Yazıcı <[email protected]> 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 Yazıcı has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Fix `HttpResponse` copyright year
test/jdk/java/net/httpclient/HttpResponseLimitingTest.java line 29:
> 27: * @test
> 28: * @bug 8328919
> 29: * @summary tests `limiting()` in
> `HttpResponse.Body{Handlers,Subscribers}`
Nit - the summary text is just free form text and doesn't get rendered in any
specific way. So the use of backticks here won't play any role.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23096#discussion_r1923488000
