-
Notifications
You must be signed in to change notification settings - Fork 4
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
Add an option to choose websocket protocol #20
Conversation
Now that the default is wss this is needed to test with local servers
Could you also update the docs in the README for this? |
Also I should probably add a |
This seems to be passing locally; not sure why CI is failing for this Merging for now; will address fixes if needed |
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.
Epic; thanks!
Published in 3.6.0 |
Now that wss is the default this change is needed to support insecure servers
Needed the tests change for the pre-commit hook to work