RFR: 8328919: Add BodyHandlers / BodySubscribers methods to handle excessive server input [v10]
Volkan Yazici
vyazici at openjdk.org
Wed Jan 22 09:45:16 UTC 2025
On Tue, 21 Jan 2025 15:17:07 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:
>> test/jdk/java/net/httpclient/HttpResponseLimitingTest.java line 70:
>>
>>> 68: import static org.junit.jupiter.api.Assertions.assertThrows;
>>> 69:
>>> 70: class HttpResponseLimitingTest {
>>
>> I think we should also add test methods to verify that the newly added methods throw a `NullPointerException` and `IllegalArgumentException` for incorrect parameters.
>
> Finally, I think we should also add a test method which verifies that a limiting BodyHandler with a specific capacity doesn't impact the headers from being delivered to the downstream/wrapped BodyHandler even if the size of the response headers exceeds the given capacity.
@jaikiran, pushed 8668a034967acd3340a30391cd5897332a068cf5 to verify the header parsing. Would you mind reviewing it and resolving this conversation if you agree with the fix, please?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23096#discussion_r1925008304
More information about the net-dev
mailing list