On Mon, 9 Nov 2026 08:47:27 GMT, Volkan Yazici <[email protected]> wrote:

>> 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.

Okay, I missed that that method adds more lines to the header list.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/31974#discussion_r4523548801

Reply via email to