[Webkit-unassigned] [Bug 81655] [WebSocket]The Sec-WebSocket-Accept MUST NOT appear more than once in an HTTP response

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Mar 20 17:26:38 PDT 2012


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





--- Comment #4 from joey <li.yin at intel.com>  2012-03-20 17:26:38 PST ---
(In reply to comment #2)
> (From update of attachment 132813 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=132813&action=review
> 
> > Source/WebCore/Modules/websockets/WebSocketHandshake.cpp:578
> > +    bool hasAcceptValue = false;
> 
> "Accept" is also an HTTP header, so this variable is named confusingly. I suggest something like "sawSecWebSocketAcceptHeaderField".

Thanks for your review, the patch has been updated already, please review again.

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



More information about the webkit-unassigned mailing list