From bdfff942f7123d41121e69bc373fb2eb5d1b7977 Mon Sep 17 00:00:00 2001
From: Christoffer Martinsson <cm@cmtec.se>
Date: Mon, 1 Dec 2025 15:16:34 +0100
Subject: [PATCH] Remove VPN external IP logging

Clean up debug logging for production.

Version: v0.1.238
---
 Cargo.lock                      |  6 +++---
 agent/Cargo.toml                |  2 +-
 agent/src/collectors/systemd.rs | 25 ++++++++-----------------
 dashboard/Cargo.toml            |  2 +-
 shared/Cargo.toml               |  2 +-
 5 files changed, 14 insertions(+), 23 deletions(-)

diff --git a/Cargo.lock b/Cargo.lock
index e5f9b7c..62ab080 100644
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -279,7 +279,7 @@ checksum = "a1d728cc89cf3aee9ff92b05e62b19ee65a02b5702cff7d5a377e32c6ae29d8d"
 
 [[package]]
 name = "cm-dashboard"
-version = "0.1.237"
+version = "0.1.238"
 dependencies = [
  "anyhow",
  "chrono",
@@ -301,7 +301,7 @@ dependencies = [
 
 [[package]]
 name = "cm-dashboard-agent"
-version = "0.1.237"
+version = "0.1.238"
 dependencies = [
  "anyhow",
  "async-trait",
@@ -325,7 +325,7 @@ dependencies = [
 
 [[package]]
 name = "cm-dashboard-shared"
-version = "0.1.237"
+version = "0.1.238"
 dependencies = [
  "chrono",
  "serde",
diff --git a/agent/Cargo.toml b/agent/Cargo.toml
index 5b1b6d3..bb2ec01 100644
--- a/agent/Cargo.toml
+++ b/agent/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "cm-dashboard-agent"
-version = "0.1.237"
+version = "0.1.238"
 edition = "2021"
 
 [dependencies]
diff --git a/agent/src/collectors/systemd.rs b/agent/src/collectors/systemd.rs
index 49f65a1..3187038 100644
--- a/agent/src/collectors/systemd.rs
+++ b/agent/src/collectors/systemd.rs
@@ -160,22 +160,15 @@ impl SystemdCollector {
                     }
 
                     if service_name == "openvpn-vpn-connection" && status_info.active_state == "active" {
-                        tracing::info!("Checking VPN external IP for service: {}", service_name);
-                        match self.get_vpn_external_ip() {
-                            Some(external_ip) => {
-                                tracing::info!("Got VPN external IP: {}", external_ip);
-                                let metrics = Vec::new();
+                        if let Some(external_ip) = self.get_vpn_external_ip() {
+                            let metrics = Vec::new();
 
-                                sub_services.push(SubServiceData {
-                                    name: format!("ip: {}", external_ip),
-                                    service_status: Status::Info,
-                                    metrics,
-                                    service_type: "vpn_route".to_string(),
-                                });
-                            }
-                            None => {
-                                tracing::warn!("Failed to get VPN external IP");
-                            }
+                            sub_services.push(SubServiceData {
+                                name: format!("ip: {}", external_ip),
+                                service_status: Status::Info,
+                                metrics,
+                                service_type: "vpn_route".to_string(),
+                            });
                         }
                     }
 
@@ -883,8 +876,6 @@ impl SystemdCollector {
             }
         }
 
-        let stderr = String::from_utf8_lossy(&output.stderr);
-        tracing::warn!("VPN external IP query failed. Exit code: {:?}, stderr: {}", output.status.code(), stderr);
         None
     }
 }
diff --git a/dashboard/Cargo.toml b/dashboard/Cargo.toml
index 04bfdd2..ccf886e 100644
--- a/dashboard/Cargo.toml
+++ b/dashboard/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "cm-dashboard"
-version = "0.1.237"
+version = "0.1.238"
 edition = "2021"
 
 [dependencies]
diff --git a/shared/Cargo.toml b/shared/Cargo.toml
index 9962dbf..c7b6111 100644
--- a/shared/Cargo.toml
+++ b/shared/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "cm-dashboard-shared"
-version = "0.1.237"
+version = "0.1.238"
 edition = "2021"
 
 [dependencies]