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