New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Only check the status code, not the full status line #2

Open

GoogleCodeExporter opened this issue Oct 12, 2015 · 0 comments

Labels

auto-migrated Priority-Medium Type-Defect

GoogleCodeExporter commented Oct 12, 2015

The WebSocket class checks the full HTTP status line of the server's response, 
while the spec explicitly says that only the status code (101) should be 
verified.

https://code.google.com/p/android-faye-client/source/browse/src/com/arguslabs/fa
yeclient/WebSocket.java#581

I just realised that if I change the text in this status line, it will break 
your client, which should not happen.

Original issue reported on code.google.com by [email protected] on 2 Apr 2014 at 11:18

The text was updated successfully, but these errors were encountered:

GoogleCodeExporter added Priority-Medium Type-Defect auto-migrated labels

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment