fix: BandwidthSpeed calculation

Author: M03ED

controller/controller.go

@@ -16,7 +16,7 @@ import (
 	"github.com/pasarguard/node/tools"
 )
 
-const NodeVersion = "0.2.3"
+const NodeVersion = "0.2.4"
 
 type Service interface {
 	Disconnect()

tools/sys.go

@@ -46,8 +46,10 @@ func GetSystemStats() (*common.SystemStatsResponse, error) {
 
 // getBandwidthSpeed returns the aggregate incoming (rx) and outgoing (tx)
 // bandwidth in bytes per second, sampled over a 1‑second interval.
+// Loopback interface (lo) is excluded from the calculation.
 func getBandwidthSpeed() (uint64, uint64, error) {
-	// 1) First snapshot
+	// 1) First snapshot with timestamp
+	start := time.Now()
 	first, err := net.IOCounters(true)
 	if err != nil {
 		return 0, 0, err
@@ -56,27 +58,48 @@ func getBandwidthSpeed() (uint64, uint64, error) {
 	// 2) Wait one second
 	time.Sleep(1 * time.Second)
 
-	// 3) Second snapshot
+	// 3) Second snapshot with timestamp
 	second, err := net.IOCounters(true)
 	if err != nil {
 		return 0, 0, err
 	}
+	end := time.Now()
 
-	// 4) Compute deltas and sum across all interfaces
-	//    Build a map from interface name → first snapshot
+	// 4) Calculate actual elapsed time (not assumed 1 second)
+	actualDuration := end.Sub(start).Seconds()
+	if actualDuration == 0 {
+		return 0, 0, nil // avoid division by zero
+	}
+
+	// 5) Build a map from interface name → first snapshot
+	// Skip loopback interface
 	prev := make(map[string]net.IOCountersStat, len(first))
 	for _, c := range first {
+		// Skip loopback interface
+		if c.Name == "lo" {
+			continue
+		}
 		prev[c.Name] = c
 	}
 
-	var totalRx, totalTx uint64
+	// 6) Compute deltas and sum across all interfaces
+	// Skip loopback interface
+	var totalRxBytes, totalTxBytes uint64
 	for _, c := range second {
+		// Skip loopback interface
+		if c.Name == "lo" {
+			continue
+		}
 		if p, ok := prev[c.Name]; ok {
-			totalRx += c.BytesRecv - p.BytesRecv
-			totalTx += c.BytesSent - p.BytesSent
+			totalRxBytes += c.BytesRecv - p.BytesRecv
+			totalTxBytes += c.BytesSent - p.BytesSent
 		}
 	}
 
-	// 5) Return the totals
-	return totalRx, totalTx, nil
+	// 7) Convert bytes to bytes per second using ACTUAL measured time
+	rxPerSecond := uint64(float64(totalRxBytes) / actualDuration)
+	txPerSecond := uint64(float64(totalTxBytes) / actualDuration)
+
+	// 8) Return the calculated rates (bytes per second)
+	return rxPerSecond, txPerSecond, nil
 }