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

Add websocket support #371

Closed
wants to merge 3 commits into from
Closed

Add websocket support #371

wants to merge 3 commits into from

Conversation

bmwant
Copy link

@bmwant bmwant commented Jul 28, 2015

Add ability to open websocket connection and send json-data to that endpoint.

@sigmavirus24
Copy link

I'm not sure websocket support belongs as a first class feature. It should either be a third-party plugin or an optional feature supported through the use of (setuptools) extras.

@jkbrzt
Copy link
Member

jkbrzt commented Sep 1, 2015

@bmwant21 thanks for your work on this 👍I agree with @sigmavirus24 that it should be a plugin. The API is not ready for this yet though, so please follow #177 for progress.

@jkbrzt jkbrzt added enhancement New feature or enhancement extensions Improve an existing feature labels Sep 1, 2015
@bmwant
Copy link
Author

bmwant commented Jul 27, 2018

hey, this branch is slightly outdated, so closing it hoping somebody will implement this as a plugin

@bmwant bmwant closed this Jul 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or enhancement extensions Improve an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants