On Wed, 4 Nov 2026 11:34:28 GMT, Michael McMahon <[email protected]> wrote:
>> Volkan Yazici has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Removed `MaxReqHeaderSizePropertyTest`
>
> test/jdk/com/sun/net/httpserver/property/MaxReqHeadersPropertyTest.java line
> 102:
>
>> 100: */
>> 101:
>> 102: class MaxReqHeadersPropertyTest {
>
> I notice all the request headers in this test have the same name "k". Not
> sure it really matters, but maybe varying them (k1, k2, ....) might be
> useful.
They already have `k1`, `k2`, etc. variations. See `createRequestHeaderLines()`
and its call-sites. Let me know if I misunderstood you.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/31974#discussion_r4510901946
