-
Notifications
You must be signed in to change notification settings - Fork 1
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
Socketio 2.0 #6
base: release
Are you sure you want to change the base?
Socketio 2.0 #6
Conversation
socketIO_client/transports.py
Outdated
except UnicodeDecodeError as error: | ||
if self.bw_comp: | ||
raise error | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
trace would be good here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some comments.
Hi, do you have any progress estimates on if and when this will get accepted? We are strugling with finding a working python client for socketio 2.x. Thanks! |
@juhhov ready for merging? |
@Hernrup have you tried this PR yet - does it work in your case? double e2e testing would be more than welcome :) |
@jupe actually no but I'll give it a try when I get to work tomorrow and let you know how it goes! |
Im sorry to say that my issues persist with the new version. Serverpython-socketio==2.0.0 Test client
Result
It immediately falls back to 1.0 and then chooses polling as transport when handshake can not be established. Some basic polling over GET seems to work
But some over post fails (wrong protocal? socketIO 1.x?)
Ill keep digging but this might provide you with some more insite. Thanks! |
Target is to support 1.x and 2.x socketio versions. Need to figure out how to detect which version server is running.