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

Attempt a reconnect if the proxy connect attempt fails #978

Merged
merged 1 commit into from Nov 13, 2016

Conversation

ToniA
Copy link
Contributor

@ToniA ToniA commented Nov 13, 2016

The 'resolver.resolve(query)' throws as exception if it cannot resolve the network address. If the network connection is lost, the resolve fails, and the exception is not catched untill the execution reaches CProxyManager::StartThread(). This will exit the proxy thread, which again is never restarted.

This is very easy to test:

  • Configure Domoticz to connect to MyDomoticz
  • Pull the network plug. After a while Domoticz gets disconnected from MyDomoticz
  • Reconnect the network. Without this change, Domoticz does not reconnect to MyDomoticz

The 'resolver.resolve(query)' throws as exception if it cannot resolve the network address. If the network connection is lost, the resolve fails, and the exception is not catched untill the execution reaches CProxyManager::StartThread(). This will exit the proxy thread, which again is never restarted.
@gizmocuz gizmocuz merged commit c3b4351 into domoticz:master Nov 13, 2016
@gizmocuz
Copy link
Contributor

Thanks!

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.

None yet

2 participants