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

Error when using multiplex.py in the multiplex example #30

Closed
lchan752 opened this issue Nov 9, 2012 · 1 comment
Closed

Error when using multiplex.py in the multiplex example #30

lchan752 opened this issue Nov 9, 2012 · 1 comment

Comments

@lchan752
Copy link

lchan752 commented Nov 9, 2012

I am having this error when self.send(msg) is called from server side.

File "/path/to/my/sockjs_server.py", line 11, in on_message
self.send(msg)
File "/path/to/my/python/lib/python2.7/site-packages/sockjs/tornado/conn.py", line 49, in send
self.session.send_message(message, binary=binary)
TypeError: send_message() got an unexpected keyword argument 'binary'

I got it to work by changing the example multiplex.py
def send_message(self, msg, binary=False): <-- I added binary=False in the argument list.
self.base.send('msg,' + self.name + ',' + msg, binary)

Is this a bug? Or am I doing anything wrong?

Thanks,
Leo

@mrjoes
Copy link
Owner

mrjoes commented Nov 9, 2012

Thanks for report, that's a bug.

Just add binary=False and ignore it in your handler. I will push the fix.

@mrjoes mrjoes closed this as completed in ed3ce99 Nov 13, 2012
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

2 participants