-
-
Notifications
You must be signed in to change notification settings - Fork 591
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
1 parent
045188c
commit ea016fa
Showing
2 changed files
with
6 additions
and
1 deletion.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ea016fa
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.
Hey Miguel, I think this update broke something. I could connect without issues to a socket.io client from a node environment. Now it doesn't work anymore. Check this SO question I created:
https://stackoverflow.com/questions/57301646/electron-socket-io-client-python-socketio-aiohttp-server-fail-to-connect
I build the same mini server in node and it works without any issue. I trying also different ports than local host 0.0.0.0 and my actual network address.
Yup, doing
pipenv install python-socketio=="4.2.1"
removes the issue.ea016fa
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.
I'm looking into it, this was also reported in miguelgrinberg/python-engineio#131.