From 4094df3a618dba0983e8cb15f580d0c7d10540bc Mon Sep 17 00:00:00 2001
From: Henry Dollman <hank@henrygd.me>
Date: Thu, 24 Oct 2024 15:10:20 -0400
Subject: [PATCH] fix: skip temperature collection if SENSORS is empty string
 (#196)

---
 beszel/internal/agent/agent.go  | 4 +++-
 beszel/internal/agent/system.go | 6 ++++--
 2 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/beszel/internal/agent/agent.go b/beszel/internal/agent/agent.go
index a902e86..6117263 100644
--- a/beszel/internal/agent/agent.go
+++ b/beszel/internal/agent/agent.go
@@ -62,7 +62,9 @@ func (a *Agent) Run(pubKey []byte, addr string) {
 	if sensors, exists := os.LookupEnv("SENSORS"); exists {
 		a.sensorsWhitelist = make(map[string]struct{})
 		for _, sensor := range strings.Split(sensors, ",") {
-			a.sensorsWhitelist[sensor] = struct{}{}
+			if sensor != "" {
+				a.sensorsWhitelist[sensor] = struct{}{}
+			}
 		}
 	}
 
diff --git a/beszel/internal/agent/system.go b/beszel/internal/agent/system.go
index 854bd93..d7ad38e 100644
--- a/beszel/internal/agent/system.go
+++ b/beszel/internal/agent/system.go
@@ -172,10 +172,12 @@ func (a *Agent) getSystemStats() system.Stats {
 	}
 
 	// temperatures (skip if sensors whitelist is set to empty string)
-	if a.sensorsWhitelist == nil || len(a.sensorsWhitelist) > 0 {
+	if a.sensorsWhitelist != nil && len(a.sensorsWhitelist) == 0 {
+		slog.Debug("Skipping temperature collection")
+	} else {
 		temps, err := sensors.TemperaturesWithContext(a.sensorsContext)
 		if err != nil {
-			// err.(*sensors.Warnings).Verbose = true
+			err.(*sensors.Warnings).Verbose = true
 			slog.Debug("Sensor error", "err", err)
 		}
 		slog.Debug("Temperature", "sensors", temps)