RFR: 8328919: Add BodyHandlers / BodySubscribers methods to handle excessive server input [v9]
Volkan Yazıcı
duke at openjdk.org
Tue Jan 21 11:03:38 UTC 2025
On Tue, 21 Jan 2025 10:42:39 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:
>> 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.
I am aware, but as a programmer, I find it more easy to read when there is a clear distinction between code and text. Unless anybody has an objection, I prefer to keep it the way it is.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23096#discussion_r1923512663
More information about the net-dev
mailing list