From 7cf475ce39f95db529a4345c989279fbf8b74c84 Mon Sep 17 00:00:00 2001 From: Jeremiah Russell Date: Fri, 10 Oct 2025 22:40:12 +0100 Subject: [PATCH] =?UTF-8?q?=F0=9F=90=9B=20fix(processor):=20handle=20None?= =?UTF-8?q?=20query=20in=20eol=5Fquery?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - fix issue where eol_query could return None, causing a panic - handle the None case by returning an error --- src/processor.rs | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/src/processor.rs b/src/processor.rs index 95af729..e6e0c24 100644 --- a/src/processor.rs +++ b/src/processor.rs @@ -37,9 +37,10 @@ impl<'a> Processor<'a> { return Err(Error::LableNotFoundInMailbox(label.to_string())); } - messages_to_trash - .message_list() - .set_query(&self.rule.eol_query()); + let Some(query) = self.rule.eol_query() else { + return Err(Error::NoQueryStringCalculated(self.rule.id())); + }; + messages_to_trash.message_list().set_query(&query); log::info!("{messages_to_trash:?}"); log::info!("Ready to run"); @@ -59,9 +60,10 @@ impl<'a> Processor<'a> { return Err(Error::LableNotFoundInMailbox(label.to_string())); } - messages_to_delete - .message_list() - .set_query(&self.rule.eol_query()); + let Some(query) = self.rule.eol_query() else { + return Err(Error::NoQueryStringCalculated(self.rule.id())); + }; + messages_to_delete.message_list().set_query(&query); log::info!("{messages_to_delete:?}"); log::info!("Ready to run");