yunhostnet / pywebsocket

Automatically exported from code.google.com/p/pywebsocket
0 stars 0 forks source link

Allow \x20 in subprotocol #30

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Purpose of code changes on this branch:
Allow \x20 in subprotocol.
The Web Socket Protocol has ben changed to allow \x20 (Space) in the 
subprotocol value.

When reviewing my code changes, please focus on:

After the review, I'll merge this branch into:
/trunk

Original issue reported on code.google.com by yuzo@chromium.org on 8 Dec 2009 at 7:46

GoogleCodeExporter commented 9 years ago
Hi, Ukai-san,

Can you review this?

Yuzo

Original comment by yuzo@chromium.org on 8 Dec 2009 at 7:47

GoogleCodeExporter commented 9 years ago
Added a test for an empty subprotocol.

In inspecting diffs, please note that there are two revisions in this change.

Yuzo

Original comment by yuzo@chromium.org on 8 Dec 2009 at 8:19

GoogleCodeExporter commented 9 years ago
Thank you for the review. I'll merge the change to the trunk.

Original comment by yuzo@chromium.org on 8 Dec 2009 at 8:30