fix logic in call to UpdatePeers

controllers/node.go

@@ -592,8 +592,8 @@ func updateNode(w http.ResponseWriter, r *http.Request) {
 		logger.Log(1, "error publishing node update"+err.Error())
 	}
 	if logic.ShouldPeersUpdate(&node, &newNode) {
-		if err := mq.UpdatePeers(&newnode) {
-		logger.Log(1, "error publishing peer update after node update"+err.Error())
+		if err := mq.UpdatePeers(&newNode); err != nil {
+			logger.Log(1, "error publishing peer update after node update"+err.Error())
 		}
 	}
 }

controllers/server_util.go

@@ -1,33 +0,0 @@
-package controller
-
-import (
-	"github.com/gravitl/netmaker/logger"
-	"github.com/gravitl/netmaker/logic"
-	"github.com/gravitl/netmaker/servercfg"
-	"github.com/gravitl/netmaker/serverctl"
-)
-
-func runServerPeerUpdate(network string, shouldPeerUpdate bool) error {
-	if servercfg.Telemetry() == "on" {
-		err := serverctl.TelemetryCheckpoint()
-		if err != nil {
-			logger.Log(1, "failed to send telemetry:", err.Error())
-		}
-	}
-	if servercfg.IsClientMode() != "on" {
-		return nil
-	}
-	var currentServerNodeID, err = logic.GetNetworkServerNodeID(network)
-	if err != nil {
-		return err
-	}
-	var currentServerNode, currErr = logic.GetNodeByID(currentServerNodeID)
-	if currErr != nil {
-		return currErr
-	}
-	if err = logic.ServerUpdate(&currentServerNode, shouldPeerUpdate); err != nil {
-		logger.Log(1, "server node:", currentServerNode.ID, "failed update")
-		return err
-	}
-	return nil
-}