Skip to content
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.

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

hybi-17 Unsupported #585

Closed
giggsey opened this issue Oct 19, 2011 · 5 comments
Closed

hybi-17 Unsupported #585

giggsey opened this issue Oct 19, 2011 · 5 comments
Labels
bug Something isn't working

Comments

@giggsey
Copy link

giggsey commented Oct 19, 2011

This just got pushed to Chrome Dev (16.0.912.0), and it appears to be using hybi-17.

Details: http://trac.webkit.org/changeset/97249

My socket.io still connects, is just falls back to another method.

@3rd-Eden
Copy link
Contributor

Ping @einaros

@einaros
Copy link
Contributor

einaros commented Oct 19, 2011

HyBi-16 to HyBi-17 has no breaking changes. Should work just fine with the current socket.io version.

@giggsey
Copy link
Author

giggsey commented Oct 19, 2011

Good point, I was on 0.8.4.

@giggsey giggsey closed this as completed Oct 19, 2011
@3rd-Eden
Copy link
Contributor

@einaros not even the Sec-WebSocket-Origin header? As we should be using that to check against the manager.get('origin') ?

@einaros
Copy link
Contributor

einaros commented Oct 19, 2011

@3rd-Eden, I've updated that as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants