From a24e7571e9cacc8ca46d04df6811288d926a99b9 Mon Sep 17 00:00:00 2001 From: Lucas Trzesniewski Date: Sun, 9 Mar 2025 23:20:01 +0100 Subject: fix: don't save empty commands (#2605) --- crates/atuin-client/src/history.rs | 9 +++++++++ 1 file changed, 9 insertions(+) (limited to 'crates') diff --git a/crates/atuin-client/src/history.rs b/crates/atuin-client/src/history.rs index 48d48eb7..6469740e 100644 --- a/crates/atuin-client/src/history.rs +++ b/crates/atuin-client/src/history.rs @@ -375,6 +375,7 @@ impl History { pub fn should_save(&self, settings: &Settings) -> bool { !(self.command.starts_with(' ') + || self.command.is_empty() || settings.history_filter.is_match(&self.command) || settings.cwd_filter.is_match(&self.cwd) || (settings.secrets_filter && SECRET_PATTERNS_RE.is_match(&self.command))) @@ -413,6 +414,13 @@ mod tests { .build() .into(); + let empty: History = History::capture() + .timestamp(time::OffsetDateTime::now_utc()) + .command("") + .cwd("/") + .build() + .into(); + let stripe_key: History = History::capture() .timestamp(time::OffsetDateTime::now_utc()) .command("curl foo.com/bar?key=sk_test_1234567890abcdefghijklmnop") @@ -436,6 +444,7 @@ mod tests { assert!(normal_command.should_save(&settings)); assert!(!with_space.should_save(&settings)); + assert!(!empty.should_save(&settings)); assert!(!stripe_key.should_save(&settings)); assert!(!secret_dir.should_save(&settings)); assert!(!with_psql.should_save(&settings)); -- cgit v1.3.1