From a92768157c2b6f295cf094e7e7cd91e62e7746e4 Mon Sep 17 00:00:00 2001 From: Niko Marmeladkov Date: Fri, 3 Jul 2026 14:10:05 +0300 Subject: [PATCH] fix hardcoded HOME paths in stop() and check_children() Both functions used format! with HOME for log paths; when daemon runs as system user without HOME, logs went to /.local/share/... Now uses state::log_path() like spawn() does. --- src/daemon.rs | 12 ++---------- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/src/daemon.rs b/src/daemon.rs index b51b887..f297bf6 100644 --- a/src/daemon.rs +++ b/src/daemon.rs @@ -222,11 +222,7 @@ fn stop(name: &str, procs: &mut HashMap) { }; let pid = p.pid; - let log_path = format!( - "{}/.local/share/rss/logs/{}.log", - std::env::var("HOME").unwrap_or_default(), - name - ); + let log_path = crate::state::log_path(name).to_string_lossy().to_string(); // stop_command if let Some(cmd) = p.cfg.resolve_stop_command(name, pid, &log_path) { @@ -276,11 +272,7 @@ fn check_children(procs: &mut HashMap) { for (name, p) in procs.iter_mut() { if let Ok(Some(status)) = p.child.try_wait() { - let log_path = format!( - "{}/.local/share/rss/logs/{}.log", - std::env::var("HOME").unwrap_or_default(), - name - ); + let log_path = crate::state::log_path(name).to_string_lossy().to_string(); let code = status .code()