fixed non ping err resulting in connected

logic/metrics/metrics.go

@@ -58,11 +58,14 @@ func Collect(iface string, peerMap models.PeerMap) (*models.Metrics, error) {
 				newMetric.Latency = 999
 			} else {
 				pingStats := pinger.Statistics()
-				newMetric.Uptime = 1
-				newMetric.Connected = true
-				newMetric.Latency = pingStats.AvgRtt.Milliseconds()
+				if pingStats.PacketsRecv > 0 {
+					newMetric.Uptime = 1
+					newMetric.Connected = true
+					newMetric.Latency = pingStats.AvgRtt.Milliseconds()
+				}
 			}
 		}
+
 		// check device peer to see if WG is working if ping failed
 		if !newMetric.Connected {
 			if currPeer.ReceiveBytes > 0 &&
@@ -72,6 +75,7 @@ func Collect(iface string, peerMap models.PeerMap) (*models.Metrics, error) {
 				newMetric.Uptime = 1
 			}
 		}
+
 		newMetric.TotalTime = 1
 		metrics.Connectivity[id] = newMetric
 	}

netclient/functions/mqpublish.go

@@ -44,7 +44,7 @@ func Checkin(ctx context.Context, wg *sync.WaitGroup) {
 		case <-ticker.C:
 			currentRun++
 			checkin(currentRun)
-			if currentRun >= 5 {
+			if currentRun >= 0 {
 				currentRun = 0
 			}
 		}
@@ -116,7 +116,7 @@ func checkin(currentRun int) {
 		}
 		Hello(&nodeCfg)
 		checkCertExpiry(&nodeCfg)
-		if currentRun >= 5 && nodeCfg.Server.Is_EE {
+		if currentRun >= 0 && nodeCfg.Server.Is_EE {
 			logger.Log(0, "collecting metrics for node", nodeCfg.Node.Name)
 			publishMetrics(&nodeCfg)
 		}