Bug 66924 – [WebSocket] update pywebsocket to 0.6b5 which support closing handshake test

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Takashi Toyoshima]

Update pywebsocket to the newest one.
Current 0.6b2 could not verify closing code and reason that a client send to the server.

[Takashi Toyoshima]

Created attachment 105140 [details]
Patch

[Yuta Kitamura]

Comment on attachment 105140 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=105140&action=review

Looks good to me. [Note: I'm not a WebKit reviewer.]

> Tools/ChangeLog:3
> +        [WebSocket] update pywebsocket to 0.6b3 which support closing handshake test

nit: support -> supports

[Takashi Toyoshima]

Created attachment 105148 [details]
Patch

[Kent Tamura]

Comment on attachment 105148 [details]
Patch

ok

[WebKit Review Bot]

Comment on attachment 105148 [details]
Patch

Clearing flags on attachment: 105148

Committed r93767: <http://trac.webkit.org/changeset/93767>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.

[Yuta Kitamura]

Unfortunately this patch caused unexpected failures on Qt bots. According to pywebsocket logs, unexpected 403 responses started to appear after this change on not only Qt bots but also other bots. Thus I rolled out this change.

http://trac.webkit.org/changeset/93776

Logs:

http://build.webkit.org/results/SnowLeopard%20Intel%20Release%20(WebKit2%20Tests)/r93773%20(14445)/pywebsocket.ws.log-25Aug2011-051651-err.txt

[2011-08-25 05:16:53,685] [WARNING] root: localhost - code 403, message Forbidden
[2011-08-25 05:16:53,710] [ERROR] root: Exception in processing request from: ('127.0.0.1', 64531)
Traceback (most recent call last):
  File "/System/Library/Frameworks/Python.framework/Versions/2.6/lib/python2.6/SocketServer.py", line 558, in process_request_thread
    self.finish_request(request, client_address)
  File "/System/Library/Frameworks/Python.framework/Versions/2.6/lib/python2.6/SocketServer.py", line 320, in finish_request
    self.RequestHandlerClass(request, client_address, self)
  File "/Volumes/Big/WebKit-BuildSlave/snowleopard-intel-release-tests-wk2/build/Tools/Scripts/webkitpy/thirdparty/autoinstalled/pywebsocket/mod_pywebsocket/standalone.py", line 381, in __init__
    self, request, client_address, server)
  File "/System/Library/Frameworks/Python.framework/Versions/2.6/lib/python2.6/SocketServer.py", line 615, in __init__
    self.handle()
  File "/System/Library/Frameworks/Python.framework/Versions/2.6/lib/python2.6/BaseHTTPServer.py", line 329, in handle
    self.handle_one_request()
  File "/System/Library/Frameworks/Python.framework/Versions/2.6/lib/python2.6/BaseHTTPServer.py", line 316, in handle_one_request
    if not self.parse_request(): # An error code has been sent, just exit
  File "/Volumes/Big/WebKit-BuildSlave/snowleopard-intel-release-tests-wk2/build/Tools/Scripts/webkitpy/thirdparty/autoinstalled/pywebsocket/mod_pywebsocket/standalone.py", line 447, in parse_request
    self.send_error(e.status)
  File "/System/Library/Frameworks/Python.framework/Versions/2.6/lib/python2.6/BaseHTTPServer.py", line 357, in send_error
    self.send_response(code, message)
  File "/System/Library/Frameworks/Python.framework/Versions/2.6/lib/python2.6/BaseHTTPServer.py", line 385, in send_response
    self.send_header('Date', self.date_time_string())
  File "/System/Library/Frameworks/Python.framework/Versions/2.6/lib/python2.6/BaseHTTPServer.py", line 390, in send_header
    self.wfile.write("%s: %s\r\n" % (keyword, value))
  File "/System/Library/Frameworks/Python.framework/Versions/2.6/lib/python2.6/socket.py", line 297, in write
    self.flush()
  File "/System/Library/Frameworks/Python.framework/Versions/2.6/lib/python2.6/socket.py", line 284, in flush
    self._sock.sendall(buffer)
error: [Errno 32] Broken pipe

[Takashi Toyoshima]

Created attachment 105483 [details]
Patch

[Kent Tamura]

Comment on attachment 105483 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=105483&action=review

> Tools/ChangeLog:3
> +        [WebSocket] update pywebsocket to 0.6b4 which supports closing handshake test

What's the difference between 0.6b2 and 0.6b3/b4?

[Takashi Toyoshima]

Created attachment 105582 [details]
Patch

[WebKit Review Bot]

Comment on attachment 105582 [details]
Patch

Clearing flags on attachment: 105582

Committed r94055: <http://trac.webkit.org/changeset/94055>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.

[Takashi Toyoshima]

Created attachment 105621 [details]
Patch

[Takashi Toyoshima]

This is the third trial.
The last change results in failure on Mac 10.5 because pywebsocket 0.6b4 used missing function in Python 2.5.1 which is the default version of Mac 10.5.

[Kent Tamura]

Comment on attachment 105621 [details]
Patch

ok, try again.

[WebKit Review Bot]

Comment on attachment 105621 [details]
Patch

Clearing flags on attachment: 105621

Committed r94146: <http://trac.webkit.org/changeset/94146>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.