You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cxf.apache.org by "Sergey Beryozkin (JIRA)" <ji...@apache.org> on 2015/04/23 14:24:38 UTC

[jira] [Commented] (CXF-6365) Cookie format written to request headers is invalid

    [ https://issues.apache.org/jira/browse/CXF-6365?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14508960#comment-14508960 ] 

Sergey Beryozkin commented on CXF-6365:
---------------------------------------

You are probably looking at a Set-Cookie section; Set-Cookie controlled by the server, but Cookie (see section 4.3.4) is controlled by a client and indeed a version goes first.
So this issue is invalid.

> Cookie format written to request headers is invalid
> ---------------------------------------------------
>
>                 Key: CXF-6365
>                 URL: https://issues.apache.org/jira/browse/CXF-6365
>             Project: CXF
>          Issue Type: Bug
>          Components: Transports
>    Affects Versions: 3.0.4
>            Reporter: Matt Kusnierz
>   Original Estimate: 5m
>  Remaining Estimate: 5m
>
> The org.apache.cxf.transport.http.Cookie.requestCookieHeader() method formats the Cookie incorrectly with the Version attribute first. The Cookie specification (RFC 2109: https://www.ietf.org/rfc/rfc2109.txt) states that the cookie name should be the first of the key-value pairs in the formatted cookie. Trying to parse cookie headers added in this way using the standard java utility: java.net.HttpCookie.parse causes an exception to be thrown: java.lang.IllegalArgumentException: Illegal cookie name.
> The fix is trivial, simply add the Version tag last instead of first. Seems to impact all versions of org.apache.cxf:cxf-rt-transports-http



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)