diff --git a/crates/erp-server/src/handlers/health.rs b/crates/erp-server/src/handlers/health.rs
index a223e0c..018102a 100644
--- a/crates/erp-server/src/handlers/health.rs
+++ b/crates/erp-server/src/handlers/health.rs
@@ -91,11 +91,14 @@ async fn check_database(db: &sea_orm::DatabaseConnection) -> ComponentStatus {
             latency_ms: Some(start.elapsed().as_millis() as u64),
             error: None,
         },
-        Err(e) => ComponentStatus {
-            status: "error".to_string(),
-            latency_ms: Some(start.elapsed().as_millis() as u64),
-            error: Some(e.to_string()),
-        },
+        Err(e) => {
+            tracing::error!(error = %e, "Database health check failed");
+            ComponentStatus {
+                status: "error".to_string(),
+                latency_ms: Some(start.elapsed().as_millis() as u64),
+                error: Some("connection failed".to_string()),
+            }
+        }
     }
 }
 
@@ -112,18 +115,24 @@ async fn check_redis(client: &redis::Client) -> ComponentStatus {
                     latency_ms: Some(start.elapsed().as_millis() as u64),
                     error: None,
                 },
-                Err(e) => ComponentStatus {
-                    status: "error".to_string(),
-                    latency_ms: Some(start.elapsed().as_millis() as u64),
-                    error: Some(e.to_string()),
-                },
+                Err(e) => {
+                    tracing::error!(error = %e, "Redis PING failed");
+                    ComponentStatus {
+                        status: "error".to_string(),
+                        latency_ms: Some(start.elapsed().as_millis() as u64),
+                        error: Some("connection failed".to_string()),
+                    }
+                }
+            }
+        }
+        Err(e) => {
+            tracing::error!(error = %e, "Redis connection failed");
+            ComponentStatus {
+                status: "error".to_string(),
+                latency_ms: Some(start.elapsed().as_millis() as u64),
+                error: Some("connection failed".to_string()),
             }
         }
-        Err(e) => ComponentStatus {
-            status: "error".to_string(),
-            latency_ms: Some(start.elapsed().as_millis() as u64),
-            error: Some(e.to_string()),
-        },
     }
 }