Jason Duell Assignee
	Description • 13 years ago

Upgrade pywebsocket to version 553.

This upgrade is needed to get binary API support in pywebsocket.

I just grabbed the whole pywebsocket tree from scratch, and made our tweaks to it--seems to work fine.

(Patrick: I didn't actually seem to tweak the exception in ConnectionTerminatedException() to get python 2.5 to work--but I'm not sure I hit it at any point.  All our WS mochitests worked fine w/o the change.  I added it in just in case.)

Added a README-MOZILLA file that describes update steps for future reference.

This should also fix bug 688768 (Python > 2.5 support).

	Jason Duell Assignee
	Comment 1 • 13 years ago

s/seem to tweak/seem to need to tweak/

	Patrick McManus [:mcmanus]
	Comment 2 • 13 years ago

> (Patrick: I didn't actually seem to tweak the exception in
> ConnectionTerminatedException() to get python 2.5 to work--but I'm not sure
> I hit it at any point.  All our WS mochitests worked fine w/o the change.  I
> added it in just in case.)
> 

iirc that change was fixing a bug that subsequently was fixed by the pywebsocket authors after I reported it, so it makes sense it might be redundant now. OTOH it might have also been related to deflate-stream support, which I just turned off on nightly. You might turn it on (via pref) to double check it still works. (just turn on the pref and run the same tests.)

	Jason Duell Assignee
	Comment 3 • 13 years ago

Passes try with or without network.websocket.extensions.stream-deflate set, so I guess let's go with the unmodified google version of _stream_base.py

	Patrick McManus [:mcmanus]
	Comment 4 • 13 years ago

Comment on attachment 562365 [details] [diff] [review]
v2 Upgrade mochitest pywebsocket server to version 553 (latest)

Review of attachment 562365 [details] [diff] [review]:
-----------------------------------------------------------------

I don't know why, but it breaks the test in 664284 (HSTS) which is something that is a must to merge.

	Jason Duell Assignee
	Comment 5 • 13 years ago

Patrick: if this still looks good other than the HSTS issue, please +r: the HSTS bug contains the breakage fix on top of this patch (seemed like the logical place to put it).

	Jason Duell Assignee
	Comment 6 • 13 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/508b748801b7

	:Ms2ger (he/him; ⌚ UTC+1/+2)
	Comment 7 • 13 years ago

https://hg.mozilla.org/mozilla-central/rev/508b748801b7