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

Current status and Tests #57

Open
jpadilla opened this issue Jan 21, 2014 · 6 comments
Open

Current status and Tests #57

jpadilla opened this issue Jan 21, 2014 · 6 comments

Comments

@jpadilla
Copy link

I see that this project has no tests only a test server.

From sockjs-twisted

SockJS-Twisted passes all SockJS-Protocol v0.3.3 tests, and all SockJS-Client qunit tests. It has been used in production environments, and should be free of any critical bugs.

What would be the status for this project? How can we make sure that sockjs-tornado passes all of SockJS-Protocol tests as well as SockJS-Client qunit tests? I could try implementing those qunit tests from sockjs-twisted and tweaking the server.py.

@mrjoes
Copy link
Owner

mrjoes commented Jan 21, 2014

Yes, sockjs-tornado is fully compatible with SockJS protocol and qunit tests.

Essentially, there's almost no activity in sockjs-tornado because it just works.

@adamboduch
Copy link

Indeed, it does just work. Any plans to support Tornado 3.x?

@jnsflint
Copy link

jnsflint commented Mar 3, 2014

We want to switch to sockjs. Does sockjs not support tornado 3.x (3.2)?

@adamboduch
Copy link

Looks like the master branch might, but I haven't had any luck with 1.0 (the pip/easy_install version).

@FZambia
Copy link
Contributor

FZambia commented Mar 3, 2014

It works like a charm with tornado >= 3.1.1

@excieve
Copy link

excieve commented Jun 16, 2014

More than a year since last release and some useful changes have gathered in master. Could it be the good time for a new release?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants