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

AssertionError: Invalid topic "disconnect:3jjhnPfKZVtle28NqGCa". #8

Closed
abdula opened this issue Oct 7, 2013 · 3 comments
Closed

AssertionError: Invalid topic "disconnect:3jjhnPfKZVtle28NqGCa". #8

abdula opened this issue Oct 7, 2013 · 3 comments
Labels

Comments

@abdula
Copy link

abdula commented Oct 7, 2013

AssertionError: Invalid topic "disconnect:3jjhnPfKZVtle28NqGCa". Topic patterns must be .-separated alphanumeric words
at check (/home/abdula/projects/node/acro-office-v2/src/acro-office/node_modules/strong-mq/lib/topic.js:34:5)
at NativePublishQueue.workerPublish as publish
at ClusterStore.publish (/home/abdula/projects/node/acro-office-v2/src/acro-office/node_modules/strong-cluster-socket.io-store/lib/cluster-store.js:62:15)
at XHRPolling.Transport.end (/home/abdula/projects/node/acro-office-v2/src/acro-office/node_modules/socket.io/lib/transport.js:469:18)

@sam-github
Copy link
Contributor

Can you confirm this fixes the problem you are seeing?

984fd7d should fix this.

We haven't released this to npmjs.org yet, but will soon.

@abdula
Copy link
Author

abdula commented Oct 8, 2013

After 984fd7d in log of socket.io:
warn - client not handshaken client should reconnect
warn - client not handshaken client should reconnect
warn - client not handshaken client should reconnect

@bajtos
Copy link
Member

bajtos commented Oct 8, 2013

After 984fd7d in log of socket.io:
warn - client not handshaken client should reconnect
warn - client not handshaken client should reconnect
warn - client not handshaken client should reconnect

This is a know bug of socket.io, a fix would require significant redesign of socket.io internals. See socketio/socket.io#952. I am afraid this issue is not going to get fixed anytime soon.

I'd recommend you setup a load-balancer that supports session affinity (sticky sessions), e.g. HAProxy.

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

No branches or pull requests

3 participants