On Mon, 20 Jan 2025 08:31:18 GMT, Volkan Yazıcı <d...@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 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