mtan93/SmartThingsPublic
1
0
Fork 0
mirror of https://github.com/mtan93/SmartThingsPublic.git synced 2026-03-27 21:04:15 +00:00
Code Issues Packages Projects Releases Wiki Activity

DVCSMP-1775 - Call poll() every 5 minutes to reduce duplicated calls (#902)

Browse Source 
* DVCSMP-1775 - Call poll() every 5 minutes to reduce duplicated calls

Call poll() every 5 minutes instead of discovery() to reduce duplicated
getActivityList() call.

* Better error logging
This commit is contained in:
Juan Pablo Risso
2016-05-19 13:58:44 -04:00
parent 9bfad5d6a4
commit b105d9d80e
1 changed files with 15 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

27
smartapps/smartthings/logitech-harmony-connect.src/logitech-harmony-connect.groovy
View File

@@ -339,7 +339,7 @@ def initialize() {
state.aux = 0 state.aux = 0
if (selectedhubs || selectedactivities) { if (selectedhubs || selectedactivities) {
addDevice() addDevice()
runEvery5Minutes("discovery") runEvery5Minutes("poll")
} }
} }
@@ -394,9 +394,9 @@ def discovery() {
} }
} catch (java.net.SocketTimeoutException e) { } catch (java.net.SocketTimeoutException e) {
log.warn "Connection to the hub timed out. Please restart the hub and try again." log.warn "Connection to the hub timed out. Please restart the hub and try again."
state.resethub = true state.resethub = true
} catch (e) { } catch (e) {
log.warn "Hostname in certificate didn't match. Please try again later." log.info "Logitech Harmony - Error: $e"
} }
return null return null
} }
@@ -474,7 +474,7 @@ def activity(dni,mode) {
def poll() { def poll() {
// GET THE LIST OF ACTIVITIES // GET THE LIST OF ACTIVITIES
if (state.HarmonyAccessToken) { if (state.HarmonyAccessToken) {
getActivityList() getActivityList()
def Params = [auth: state.HarmonyAccessToken] def Params = [auth: state.HarmonyAccessToken]
def url = "https://home.myharmony.com/cloudapi/state?${toQueryString(Params)}" def url = "https://home.myharmony.com/cloudapi/state?${toQueryString(Params)}"
try { try {
@@ -520,14 +520,17 @@ def poll() {
return "Poll completed $map - $state.hubs" return "Poll completed $map - $state.hubs"
} }
} catch (groovyx.net.http.HttpResponseException e) { } catch (groovyx.net.http.HttpResponseException e) {
if (e.statusCode == 401) { // token is expired if (e.statusCode == 401) { // token is expired
state.remove("HarmonyAccessToken") state.remove("HarmonyAccessToken")
return "Harmony Access token has expired" log.warn "Harmony Access token has expired"
} }
} catch(Exception e) { } catch (java.net.SocketTimeoutException e) {
log.trace e log.warn "Connection to the hub timed out. Please restart the hub and try again."
} state.resethub = true
} } catch (e) {
log.info "Logitech Harmony - Error: $e"
}
}
} }