[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