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

Automatically reconnect on idle, fixes #34 #39

Merged
merged 3 commits into from
Oct 14, 2023

Conversation

tris
Copy link
Collaborator

@tris tris commented Oct 12, 2023

Should be pretty self-explanatory.

Tested locally:

  1. Run ecoflow_exporter.py
  2. Wait 10 seconds
  3. Observe that events are flowing
  4. Open EcoFlow mobile app
  5. Wait 10 seconds
  6. Observe that events are still flowing
  7. Close EcoFlow mobile app
  8. Wait 10 seconds
  9. Observe that events stop (Message queue is empty)
  10. Wait 60 seconds
  11. Observe that the client reconnects to MQTT
  12. Wait 10 seconds
  13. Observe that events start flowing again

@berezhinskiy
Copy link
Owner

@tris Thanks for contributing. I believe the MQTT_TIMEOUT environment variable will be beneficial. I'll investigate the issue with the CI Docker build.

@berezhinskiy berezhinskiy self-requested a review October 12, 2023 07:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants