summaryrefslogtreecommitdiffstats
path: root/matterclient
diff options
context:
space:
mode:
authorDuco van Amstel <duco.vanamstel@gmail.com>2018-12-08 16:04:10 +0000
committerWim <wim@42.be>2018-12-08 17:04:10 +0100
commita2754f15fc2bc2eede94e2e41712f9f0585f2426 (patch)
tree52894a30d0ca7a06b90a7b2266fde41a5e8404c5 /matterclient
parentb6d81f34ba7ee88e7a18a27cf4e0aeebaaf82c79 (diff)
downloadmatterbridge-msglm-a2754f15fc2bc2eede94e2e41712f9f0585f2426.tar.gz
matterbridge-msglm-a2754f15fc2bc2eede94e2e41712f9f0585f2426.tar.bz2
matterbridge-msglm-a2754f15fc2bc2eede94e2e41712f9f0585f2426.zip
Enable errcheck linter (#646)
Diffstat (limited to 'matterclient')
-rw-r--r--matterclient/helpers.go3
-rw-r--r--matterclient/matterclient.go4
-rw-r--r--matterclient/messages.go8
3 files changed, 10 insertions, 5 deletions
diff --git a/matterclient/helpers.go b/matterclient/helpers.go
index 05497311..625fffaa 100644
--- a/matterclient/helpers.go
+++ b/matterclient/helpers.go
@@ -263,8 +263,7 @@ func (m *MMClient) sendWSRequest(action string, data map[string]interface{}) err
req.Data = data
m.WsSequence++
m.log.Debugf("sendWsRequest %#v", req)
- m.WsClient.WriteJSON(req)
- return nil
+ return m.WsClient.WriteJSON(req)
}
func supportedVersion(version string) bool {
diff --git a/matterclient/matterclient.go b/matterclient/matterclient.go
index 4997011b..f15b1d1b 100644
--- a/matterclient/matterclient.go
+++ b/matterclient/matterclient.go
@@ -208,7 +208,9 @@ func (m *MMClient) StatusLoop() {
return
}
if m.WsConnected {
- m.checkAlive()
+ if err := m.checkAlive(); err != nil {
+ log.Errorf("Connection is not alive: %#v", err)
+ }
select {
case <-m.WsPingChan:
m.log.Debug("WS PONG received")
diff --git a/matterclient/messages.go b/matterclient/messages.go
index 28e3ec28..985cfe04 100644
--- a/matterclient/messages.go
+++ b/matterclient/messages.go
@@ -53,7 +53,9 @@ func (m *MMClient) parseMessage(rmsg *Message) {
m.UpdateUser(user["id"].(string))
}
case "group_added":
- m.UpdateChannels()
+ if err := m.UpdateChannels(); err != nil {
+ m.log.Errorf("failed to update channels: %#v", err)
+ }
/*
case model.ACTION_USER_REMOVED:
m.handleWsActionUserRemoved(&rmsg)
@@ -186,7 +188,9 @@ func (m *MMClient) SendDirectMessageProps(toUserId string, msg string, props map
channelName := model.GetDMNameFromIds(toUserId, m.User.Id)
// update our channels
- m.UpdateChannels()
+ if err := m.UpdateChannels(); err != nil {
+ m.log.Errorf("failed to update channels: %#v", err)
+ }
// build & send the message
msg = strings.Replace(msg, "\r", "", -1)