[Webkit-unassigned] [Bug 115863] [WebSocket] Update pywebsocket to r760

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed May 15 17:07:53 PDT 2013


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





--- Comment #8 from Lamarque V. Souza <Lamarque.Souza at basyskom.com>  2013-05-15 17:06:19 PST ---
(In reply to comment #7)
> (In reply to comment #5)
> > (In reply to comment #3)
> > > Wait, why is this not supported? This has nothing to do with compression, please explain.
> > 
> > Blink's developers deleted them because they failed after pywebsocket update. I have just researched and the tests were restored in a later commit [1]. I need to figure out what changes are necessary to make them work properly (they still does not work here).
> 
> Yes. They were originally testing the Sec-WebSocket-Location header of HyBi 00 that no longer exists in RFC 6455. As side effect, they were testing that the string passed to the WebSocket constructor as url argument are correctly parsed (in order to build the Sec-WebSocket-Location header). To keep this code path tested for RFC 6455 protocol, too, we were setting the ws_location attribute of _StandaloneRequest object using one manually built from the host header and the resource inside the _wsh.py file.

Thank you for the explanation.

> Since Chromium no longer supports HyBi 00, we've cleaned up these Sec-WebSocket-Location related files and code, and added back some new layout test items that directly check the host header and the resource.

The remove of support for HyBi 00 was done before the WebKit fork (or Blink was created), correct? The last mention of Sec-WebSocket-Location in WebKit is from last year by what I can see in git's log.

-- 
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