Add debug logging for filesystem discovery

Fix filesystem metric parsing bounds error in dashboard
Prevent string slicing panic in extract_filesystem_metric when parsing individual filesystem metrics. This resolves the issue where filesystem entries show —% —GB/—GB instead of actual usage.
2025-11-23 15:26:49 +01:00 · 2025-11-23 15:23:15 +01:00
6 changed files with 17 additions and 8 deletions
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -279,7 +279,7 @@ checksum = "a1d728cc89cf3aee9ff92b05e62b19ee65a02b5702cff7d5a377e32c6ae29d8d"

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

 [[package]]
 name = "cm-dashboard-agent"
-version = "0.1.113"
+version = "0.1.115"
 dependencies = [
 "anyhow",
 "async-trait",
@@ -324,7 +324,7 @@ dependencies = [

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

 [dependencies]
--- a/agent/src/collectors/disk.rs
+++ b/agent/src/collectors/disk.rs
@@ -665,6 +665,11 @@ impl DiskCollector {
        });

        // Individual filesystem metrics
+        debug!("Drive {} has {} filesystems", drive_name, drive.filesystems.len());
+        for filesystem in &drive.filesystems {
+            debug!("Filesystem {}: {} bytes used / {} bytes total", 
+                filesystem.mount_point, filesystem.used_bytes, filesystem.total_bytes);
+        }
        for filesystem in &drive.filesystems {
            let fs_name = if filesystem.mount_point == "/" {
                "root".to_string()
--- a/dashboard/Cargo.toml
+++ b/dashboard/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "cm-dashboard"
-version = "0.1.113"
+version = "0.1.115"
 edition = "2021"

 [dependencies]
--- a/dashboard/src/ui/widgets/system.rs
+++ b/dashboard/src/ui/widgets/system.rs
@@ -391,8 +391,12 @@ impl SystemWidget {
                
                for suffix in known_suffixes {
                    if after_fs.ends_with(suffix) {
-                        let fs_name = after_fs[..after_fs.len() - suffix.len() - 1].to_string(); // Remove suffix + underscore
-                        return (Some(fs_name), Some(suffix.to_string()));
+                        // Safely calculate fs_name length
+                        let suffix_with_underscore = suffix.len() + 1;
+                        if after_fs.len() > suffix_with_underscore {
+                            let fs_name = after_fs[..after_fs.len() - suffix_with_underscore].to_string();
+                            return (Some(fs_name), Some(suffix.to_string()));
+                        }
                    }
                }
            }
--- a/shared/Cargo.toml
+++ b/shared/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "cm-dashboard-shared"
-version = "0.1.113"
+version = "0.1.115"
 edition = "2021"

 [dependencies]
Author	SHA1	Message	Date
Christoffer Martinsson	07422cd0a7	Add debug logging for filesystem discovery All checks were successful Build and Release / build-and-release (push) Successful in 1m18s Details	2025-11-23 15:26:49 +01:00
Christoffer Martinsson	de30b80219	Fix filesystem metric parsing bounds error in dashboard All checks were successful Build and Release / build-and-release (push) Successful in 1m8s Details Prevent string slicing panic in extract_filesystem_metric when parsing individual filesystem metrics. This resolves the issue where filesystem entries show —% —GB/—GB instead of actual usage.	2025-11-23 15:23:15 +01:00