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

Command retry on connection lost #17

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

Conversation

orenskl
Copy link

@orenskl orenskl commented Apr 29, 2021

I added a command resend mechanism to allow on/off of the receiver in case the connection is lost

@Hyralex
Copy link
Collaborator

Hyralex commented Jul 14, 2021

What is the reason to add this mechanism? The library is reconnecting itself as soon as the TCP connection is lost. Is it that the connection was lost or that the command failed to execute the first time?

@orenskl
Copy link
Author

orenskl commented Jul 14, 2021

I will try to describe the problem I was trying to solve. I tried to connect to my MRX from two separate end points, one is the home assistant and one is the MRX iOS app. Once I connect from home assistant the TCP connection stays ok, but when I turn on the iOS app the connect the previous connect to the home assistant gets lost. If I now turn on the MRX from the home assistant I get an error since the connection is lost.

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.

2 participants