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

Reset _stream_ready after exception in _make_connection. #2

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jprouty
Copy link

@jprouty jprouty commented Oct 25, 2022

Please consider this PR. I am experiencing this issue when running homeassistant. I'm not sure why my myLink is having intermittent connectivity issues (maybe I am exacerbating it with my automation frequency). This PR fixes HA losing control of the blinds once encountering an error (ie, subsequent attempts to control the blind can often succeed).

Reset `_stream_ready` Event after an exception in `_make_connection`. This allows for subsequent blind control to succeed after a network failure. Otherwise (as it stands today), all future attempts to control a blind result in an infinite wait in _make_connection#79 on `await self._stream_ready.wait().`
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