cxf-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sergey Beryozkin (JIRA)" <j...@apache.org>
Subject [jira] Commented: (CXF-3035) Missing Cookies Due to org.apache.cxf.jaxrs.client.AbstractClient#setResponseBuilder(HttpURLConnection, Exchange) line 341
Date Tue, 05 Oct 2010 21:55:35 GMT

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

Sergey Beryozkin commented on CXF-3035:
---------------------------------------

Hi - thanks for the 2nd patch.
I have a test checking for Allow which has the following format :
Allow: POST,GET

So this change breaks the test expecting the list of 2 values in this example.

Yeah I can see one of those cookies contains a comma in its Expires value;
JAXRS NewCookie does not even support that Expires thing...

What I'll do is that I'll provide a workaround. i.e, I'd explcitly check if it's a SetCookie
and then set it as is, otherwise - continue splitting

> Missing Cookies Due to org.apache.cxf.jaxrs.client.AbstractClient#setResponseBuilder(HttpURLConnection,
Exchange) line 341
> --------------------------------------------------------------------------------------------------------------------------
>
>                 Key: CXF-3035
>                 URL: https://issues.apache.org/jira/browse/CXF-3035
>             Project: CXF
>          Issue Type: Bug
>          Components: JAX-RS
>    Affects Versions: 2.2.10
>         Environment: any
>            Reporter: Joachim F. Kainz
>            Assignee: Sergey Beryozkin
>             Fix For: 2.3, 2.2.11
>
>         Attachments: cxf-rt-frontend-jaxrs-2.2.10.1.patch
>
>
> If you try wget -S www.walmart.com you can see that our tomcat server returns a header
which contains "Set-Cookie:" multiple times:
> jfk@preding:/tmp$ wget -S www.walmart.com
> --2010-10-04 16:44:22--  http://www.walmart.com/
> Resolving www.walmart.com... 161.170.244.20
> Connecting to www.walmart.com|161.170.244.20|:80... connected.
> HTTP request sent, awaiting response... 
>   HTTP/1.1 200 OK
>   Date: Mon, 04 Oct 2010 23:44:22 GMT
>   Server: Apache/2.2.15
>   Via: HTTP/1.0 nw106 (nw106_6799663104_56977920)
>   Set-Cookie: cef.env=PROD; Domain=.walmart.com; Path=/
>   Set-Cookie: com.wm.visitor=10789493347; Domain=.walmart.com; Expires=Thu, 01-Oct-2020
23:44:22 GMT; Path=/
>   Set-Cookie: spcf.backup="|com.wm.visitor:10789493347|"; Version=1; Domain=.walmart.com;
Path=/
>   Set-Cookie: com.wm.anoncart=107894933471602436; Domain=.walmart.com; Expires=Thu, 01-Oct-2020
23:44:22 GMT; Path=/
>   Set-Cookie: spcf.backup="|com.wm.anoncart:107894933471602436|:|com.wm.visitor:10789493347|";
Version=1; Domain=.walmart.com; Path=/
>   Set-Cookie: WMSessionID=0000000456ff25583fcc12af37682dc03fb73f1623a00e60_1286235862759_SSL206_10-15-43-137_1286235862759_10.9_N_;
Domain=.walmart.com; Path=/
>   Set-Cookie: cef.env=PROD+B++H++D++Y+%3Fcat%3D3891+C+; Domain=.walmart.com; Path=/
>   Set-Cookie: com.wm.reflector="reflectorid:0000000000000000000000@lastupd:1286235862786@firstcreate:1286235862786";
Version=1; Domain=.walmart.com; Path=/
>   Cache-Control: private
>   Cache-Control: max-age=900
>   Last-Modified: Mon, 04 Oct 2010 23:44:22 GMT
>   Expires: Mon, 04 Oct 2010 23:44:22 GMT
>   Set-Cookie: prefses=NAVPATH~10DcurrentDS0A0|1A0; Domain=.walmart.com; Path=/
>   Vary: Accept-Encoding
>   Connection: close
>   Content-Type: text/html;ISO-8859-1;charset=ISO-8859-1
>   Set-Cookie: NSC_xxx.xbmnbsu.dpn-mc=445c13ab3660;path=/
> Length: unspecified [text/html]
> Saving to: `index.html'
>     [   <=>                                                                   
                                                             ] 83,335       132K/s   in 0.6s
   
> 2010-10-04 16:44:23 (132 KB/s) - `index.html' saved [83335]
> Unfortunately, everything but the first "Set-Cookie" is stripped by line 341 in org.apache.cxf.jaxrs.client.AbstractClient#setResponseBuilder(HttpURLConnection,
Exchange)
> Instead of the current code, which only uses the first value returned by getValue(),
it would be great if the next version could iterate through every item in the list.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message