From ead6a2698f9d5c5c8fc16a2e44ae1f299a432d0f Mon Sep 17 00:00:00 2001 From: Sarah Jamie Lewis Date: Sun, 31 Oct 2021 12:16:50 -0700 Subject: [PATCH] JoinServer errors should result in Error log --- peer/cwtch_peer.go | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/peer/cwtch_peer.go b/peer/cwtch_peer.go index d4f8bff..bdf78ff 100644 --- a/peer/cwtch_peer.go +++ b/peer/cwtch_peer.go @@ -706,8 +706,9 @@ func (cp *cwtchPeer) StartServerConnections() { for _, contact := range cp.GetContacts() { if cp.GetContact(contact).IsServer() { err := cp.JoinServer(contact) - if err == nil { - + if err != nil { + // Almost certainly a programming error so print it.. + log.Errorf("error joining server %v", err) } } } @@ -853,8 +854,8 @@ func (cp *cwtchPeer) eventHandler() { // Automated Join Server Request triggered by a plugin. log.Debugf("profile received an automated retry event for %v", ev.Data[event.GroupServer]) err := cp.JoinServer(ev.Data[event.GroupServer]) - if err == nil { - log.Debugf("error joining server... %v", err) + if err != nil { + log.Errorf("error joining server... %v", err) } case event.NewGetValMessageFromPeer: onion := ev.Data[event.RemotePeer]