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 reconnect support to snapcast server control. #17

Merged
merged 1 commit into from
Nov 16, 2017

Conversation

janLo
Copy link
Contributor

@janLo janLo commented Nov 16, 2017

This adds a new parameter "reconnect" to the Snapserver instance that
can be set to true, to make the instance trying to reconnect to the
server if the connection was lost.

The reconnect is tried immediately. If not successful, it will retry
every 5 seconds.

This is useful when you use the Snapserver implementation in some
long-running application (like HomeAssistant) and want to be able to
use the snapcast integration after a reboot/restart of the snapcast
server without a need to start the using application.

(see home-assistant/core#10264)

This adds a new parameter "reconnect" to the Snapserver instance that
can be set to true, to make the instance trying to reconnect to the
server if the connection was lost.

The reconnect is tried immediately. If not successful, it will retry
every 5 seconds.

Signed-off-by: Jan Losinski <losinski@wh2.tu-dresden.de>
@happyleavesaoc happyleavesaoc merged commit ae2d75c into happyleavesaoc:master Nov 16, 2017
@happyleavesaoc
Copy link
Owner

Nice feature, thanks!

@janLo
Copy link
Contributor Author

janLo commented Nov 16, 2017

Can you release a new version with it? I'll make sure that it will bumped in home assistant.

@happyleavesaoc
Copy link
Owner

Done. 2.0.8 pushed to pypi.

@janLo
Copy link
Contributor Author

janLo commented Nov 17, 2017

Thank you!

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