RFR: 8276774: Cookie stored in CookieHandler not sent if user headers contain cookie [v4]
Daniel Fuchs
dfuchs at openjdk.java.net
Fri Nov 19 10:12:23 UTC 2021
On Fri, 19 Nov 2021 02:44:52 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:
>> Daniel Fuchs has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains six additional commits since the last revision:
>>
>> - Merge branch 'master' into user-cookie-8276774
>> - Improved comment as requested during review
>> - More cleanup
>> - Minor cleanup
>> - Updated tests
>> - 8276774: Cookie stored in CookieHandler not sent if user headers contain cookie
>
> src/java.net.http/share/classes/jdk/internal/net/http/Stream.java line 659:
>
>> 657:
>> 658: // Filter any headers from systemHeaders that are set in userHeaders
>> 659: // except for "Cookies" - user cookies will be appended to system
>
> Hello Daniel, A minor comment issue - "Cookie" instead of "Cookies", perhaps?
Changed to "Cookie:"
> test/jdk/java/net/httpclient/UserCookieTest.java line 2:
>
>> 1: /*
>> 2: * Copyright (c) 2018, Oracle and/or its affiliates. All rights reserved.
>
> Should the year be 2021?
Darn. I thought I'd already fixed that. Good catch!
-------------
PR: https://git.openjdk.java.net/jdk/pull/6408
More information about the net-dev
mailing list