[Webkit-unassigned] [Bug 82041] [BlackBerry]Missing cookies from HTTP response header in Network tab of Web Inspector.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=82041





--- Comment #16 from Jason <jason.liu at torchmobile.com.cn>  2012-03-28 00:09:42 PST ---
(In reply to comment #15)
> (In reply to comment #13)
> > (In reply to comment #12)
> > > (In reply to comment #11)
> > > > (In reply to comment #10)
> > > > > > Source/WebCore/platform/network/blackberry/NetworkJob.cpp:319
> > > > > > +            m_response.setHTTPHeaderField(key, m_response.httpHeaderField(key) + "\r\n" + value);
> > > > > 
> > > > > I still would like to know about the delimiter, as Leo mentioned.  Can you tell us whether the delimiter of \r\n is a good choice and if other ports use it?
> > > > 
> > > > I am compiling QT. Trying to find QT's implementation.
> > > 
> > > Qt uses comma(",") to separate them.
> > 
> > Where does qt porting do this? Network stack? or WebCore?
> 
> Network stack.

"\r\n" or "," (In reply to comment #14)
> (In reply to comment #12)
> > (In reply to comment #11)
> > > (In reply to comment #10)
> > > > > Source/WebCore/platform/network/blackberry/NetworkJob.cpp:319
> > > > > +            m_response.setHTTPHeaderField(key, m_response.httpHeaderField(key) + "\r\n" + value);
> > > > 
> > > > I still would like to know about the delimiter, as Leo mentioned.  Can you tell us whether the delimiter of \r\n is a good choice and if other ports use it?
> > > 
> > > I am compiling QT. Trying to find QT's implementation.
> > 
> > Qt uses comma(",") to separate them.
> 
> But Qt's inspector doesn't show ",".
> Qt's inspector:
> Set-Cookie:cookieONE=ONE; expires=Tue, 27-Mar-2012 04:11:32 GMT
> cookieTWO=TWO; expires=Tue, 27-Mar-2012 04:11:32 GMT
> cookieTHREE=THREE; expires=Tue, 27-Mar-2012 04:11:32 GMT
> cookieFOUR=FOUR; expires=Tue, 27-Mar-2012 04:11:32 GMT
> 
> Playbook's inspector(using ","):
> Set-Cookie:cookieONE=ONE; expires=Tue, 27-Mar-2012 04:09:22 GMT,cookieTWO=TWO; expires=Tue, 27-Mar-2012 04:09:22 GMT,cookieTHREE=THREE; expires=Tue, 27-Mar-2012 04:09:22 GMT,cookieFOUR=FOUR; expires=Tue, 27-Mar-2012 04:09:22 GMT
> 
> I think "\r\n" looks better for our code.

"\r\n" or "," only affect inspector's showing.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.