[Webkit-unassigned] [Bug 117342] [curl] Merge http response header values

[bugzilla-daemon at webkit.org]

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





--- Comment #6 from Peter Gal <galpeter at inf.u-szeged.hu>  2013-06-10 00:38:33 PST ---
(In reply to comment #5)
> (From update of attachment 204033 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=204033&action=review
> 
> > Source/WebCore/platform/network/curl/ResourceHandleManager.cpp:320
> > +            d->m_response.addHTTPHeaderField(header.left(splitPos), header.substring(splitPos+1).stripWhiteSpace());
> 
> Are there ever cases where we do want to replace the HTTP header field, rather than appending?

I was wondering on that also, so I've re-read the rfc multiple times and as far as I can see most of the headers are allowed to be merged. If I remember correctly libsoup merges all HTTP headers. In the blackberry they have a white list to specify which headers can be merged.

Should I do something like the blackberry guys and add a white list?

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