diff options
author | Duco van Amstel <duco.vanamstel@gmail.com> | 2018-12-08 16:04:10 +0000 |
---|---|---|
committer | Wim <wim@42.be> | 2018-12-08 17:04:10 +0100 |
commit | a2754f15fc2bc2eede94e2e41712f9f0585f2426 (patch) | |
tree | 52894a30d0ca7a06b90a7b2266fde41a5e8404c5 /matterclient | |
parent | b6d81f34ba7ee88e7a18a27cf4e0aeebaaf82c79 (diff) | |
download | matterbridge-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.go | 3 | ||||
-rw-r--r-- | matterclient/matterclient.go | 4 | ||||
-rw-r--r-- | matterclient/messages.go | 8 |
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) |