From a841a1a6fedc9ecf8b84b2c171827ac7243a1443 Mon Sep 17 00:00:00 2001
From: rcourtman <courtmanr@gmail.com>
Date: Sun, 19 Oct 2025 14:06:18 +0000
Subject: [PATCH] fix: show success message instead of warning when using
 pulse-sensor-proxy
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

When the setup script detects TEMPERATURE_PROXY_KEY (proxy is available),
it now shows a clear success message instead of attempting SSH verification.

The verification check doesn't work with proxy-based setups since the
container doesn't have SSH keys - all temperature collection happens via
the Unix socket to pulse-sensor-proxy, which handles SSH.

Now shows:
✓ Temperature monitoring configured via pulse-sensor-proxy
  Temperature data will appear in the dashboard within 10 seconds

Instead of the misleading:
⚠️  Unable to verify SSH connectivity.
   Temperature data will appear once SSH connectivity is configured.
---
 internal/api/config_handlers.go | 19 +++++++++----------
 1 file changed, 9 insertions(+), 10 deletions(-)

diff --git a/internal/api/config_handlers.go b/internal/api/config_handlers.go
index ec6ee45..3e42fff 100644
--- a/internal/api/config_handlers.go
+++ b/internal/api/config_handlers.go
@@ -4168,16 +4168,15 @@ EOF
 
                     # Verify that Pulse can actually SSH to the configured nodes
                     echo ""
-                    # Only verify connectivity if not containerized or if proxy is installed
-                    # Containerized Pulse without proxy cannot directly SSH to nodes
-                    SHOULD_VERIFY_SSH=false
-                    if [ "$PULSE_IS_CONTAINERIZED" != true ]; then
-                        SHOULD_VERIFY_SSH=true
-                    elif systemctl is-active --quiet pulse-sensor-proxy 2>/dev/null; then
-                        SHOULD_VERIFY_SSH=true
-                    fi
-
-                    if [ "$SHOULD_VERIFY_SSH" = true ]; then
+                    # Check if we're using the temperature proxy
+                    # If proxy key was detected earlier, we're using proxy-based temperature monitoring
+                    if [ -n "$TEMPERATURE_PROXY_KEY" ]; then
+                        # Using proxy - verification not needed, proxy handles SSH
+                        echo "✓ Temperature monitoring configured via pulse-sensor-proxy"
+                        echo "  Temperature data will appear in the dashboard within 10 seconds"
+                        echo ""
+                    elif [ "$PULSE_IS_CONTAINERIZED" != true ]; then
+                        # Non-containerized Pulse - can verify SSH directly
                         echo "Verifying temperature monitoring connectivity from Pulse..."
                         echo ""