Skip to content

fixed websocket issue - #1

Open
RandomLambda wants to merge 1 commit into
SilverBeavers:mainfrom
RandomLambda:main
Open

fixed websocket issue#1
RandomLambda wants to merge 1 commit into
SilverBeavers:mainfrom
RandomLambda:main

Conversation

@RandomLambda

Copy link
Copy Markdown

fixed compatibility issue with websocket. the open property was no longer available.

as per their documentation:

https://websockets.readthedocs.io/en/stable/reference/legacy/client.html#websockets.legacy.client.WebSocketClientProtocol.open
True when the connection is open; False otherwise.

This attribute may be used to detect disconnections. However, this approach is discouraged per the [EAFP](https://docs.python.org/3/glossary.html#term-eafp) principle. Instead, you should handle [ConnectionClosed](https://websockets.readthedocs.io/en/stable/reference/exceptions.html#websockets.exceptions.ConnectionClosed) exceptions.

fixed compatibility issue with websocket
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

Successfully merging this pull request may close these issues.

1 participant