Remove status icon from VPN IP and change to lowercase

Change display from "IP: X.X.X.X" to "ip: X.X.X.X". Remove status icon for vpn_route service type. Version: v0.1.236
Add info-level logging for VPN external IP debugging
2025-12-01 14:40:21 +01:00 · 2025-12-01 14:30:25 +01:00 · 2025-12-01 14:14:41 +01:00
6 changed files with 41 additions and 17 deletions
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -279,7 +279,7 @@ checksum = "a1d728cc89cf3aee9ff92b05e62b19ee65a02b5702cff7d5a377e32c6ae29d8d"

 [[package]]
 name = "cm-dashboard"
-version = "0.1.233"
+version = "0.1.236"
 dependencies = [
 "anyhow",
 "chrono",
@@ -301,7 +301,7 @@ dependencies = [

 [[package]]
 name = "cm-dashboard-agent"
-version = "0.1.233"
+version = "0.1.236"
 dependencies = [
 "anyhow",
 "async-trait",
@@ -325,7 +325,7 @@ dependencies = [

 [[package]]
 name = "cm-dashboard-shared"
-version = "0.1.233"
+version = "0.1.236"
 dependencies = [
 "chrono",
 "serde",
--- a/agent/Cargo.toml
+++ b/agent/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "cm-dashboard-agent"
-version = "0.1.233"
+version = "0.1.236"
 edition = "2021"

 [dependencies]
--- a/agent/src/collectors/systemd.rs
+++ b/agent/src/collectors/systemd.rs
@@ -159,16 +159,23 @@ impl SystemdCollector {
                        }
                    }

-                    if service_name.contains("openvpn-vpn-connection") && status_info.active_state == "active" {
-                        if let Some(external_ip) = self.get_vpn_external_ip() {
-                            let metrics = Vec::new();
+                    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();

-                            sub_services.push(SubServiceData {
-                                name: format!("IP: {}", external_ip),
-                                service_status: Status::Ok,
-                                metrics,
-                                service_type: "vpn_route".to_string(),
-                            });
+                                sub_services.push(SubServiceData {
+                                    name: format!("ip: {}", external_ip),
+                                    service_status: Status::Ok,
+                                    metrics,
+                                    service_type: "vpn_route".to_string(),
+                                });
+                            }
+                            None => {
+                                tracing::warn!("Failed to get VPN external IP");
+                            }
                        }
                    }

@@ -855,6 +862,7 @@ impl SystemdCollector {
        let output = Command::new("timeout")
            .args(&[
                "5",
+                "sudo",
                "ip",
                "netns",
                "exec",
@@ -871,12 +879,12 @@ impl SystemdCollector {
        if output.status.success() {
            let ip = String::from_utf8_lossy(&output.stdout).trim().to_string();
            if !ip.is_empty() && ip.contains('.') {
-                debug!("VPN external IP: {}", ip);
                return Some(ip);
            }
        }

-        debug!("Failed to get VPN external IP");
+        let stderr = String::from_utf8_lossy(&output.stderr);
+        tracing::warn!("VPN external IP query failed. Exit code: {:?}, stderr: {}", output.status.code(), stderr);
        None
    }
 }
--- a/dashboard/Cargo.toml
+++ b/dashboard/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "cm-dashboard"
-version = "0.1.233"
+version = "0.1.236"
 edition = "2021"

 [dependencies]
--- a/dashboard/src/ui/widgets/services.rs
+++ b/dashboard/src/ui/widgets/services.rs
@@ -298,6 +298,22 @@ impl ServicesWidget {
                        .bg(Theme::background()),
                ),
            ]
+        } else if info.service_type == "vpn_route" {
+            // VPN route info - no status icon
+            vec![
+                // Indentation and tree prefix
+                ratatui::text::Span::styled(
+                    format!("  {} ", tree_symbol),
+                    Typography::tree(),
+                ),
+                // Service name (no icon)
+                ratatui::text::Span::styled(
+                    short_name,
+                    Style::default()
+                        .fg(Theme::secondary_text())
+                        .bg(Theme::background()),
+                ),
+            ]
        } else {
            vec![
                // Indentation and tree prefix
--- a/shared/Cargo.toml
+++ b/shared/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "cm-dashboard-shared"
-version = "0.1.233"
+version = "0.1.236"
 edition = "2021"

 [dependencies]
Author	SHA1	Message	Date
Christoffer Martinsson	966ba27b1e	Remove status icon from VPN IP and change to lowercase All checks were successful Build and Release / build-and-release (push) Successful in 1m19s Details Change display from "IP: X.X.X.X" to "ip: X.X.X.X". Remove status icon for vpn_route service type. Version: v0.1.236	2025-12-01 14:40:21 +01:00
Christoffer Martinsson	6c6c9144bd	Add info-level logging for VPN external IP debugging All checks were successful Build and Release / build-and-release (push) Successful in 1m10s Details Change debug to info/warn logging to diagnose VPN IP query issues. Use exact service name match instead of contains. Version: v0.1.235	2025-12-01 14:30:25 +01:00
Christoffer Martinsson	3fdcec8047	Add sudo for VPN namespace access All checks were successful Build and Release / build-and-release (push) Successful in 1m10s Details Use sudo to access vpn namespace for external IP query. Requires corresponding sudo permission in NixOS config. Version: v0.1.234	2025-12-01 14:14:41 +01:00