From 21703b9f8ecf816ba7a52efcfb7dd4024e40599b Mon Sep 17 00:00:00 2001
From: James Magahern <james@magahern.com>
Date: Fri, 27 Jun 2025 00:52:09 -0700
Subject: [PATCH] AttachmentStore: less chatty logging

---
 kordophoned/src/daemon/attachment_store.rs | 10 +++++-----
 kordophoned/src/daemon/mod.rs              |  4 ++--
 kordophoned/src/daemon/update_monitor.rs   |  4 ++--
 3 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/kordophoned/src/daemon/attachment_store.rs b/kordophoned/src/daemon/attachment_store.rs
index 8158c6a..c699503 100644
--- a/kordophoned/src/daemon/attachment_store.rs
+++ b/kordophoned/src/daemon/attachment_store.rs
@@ -120,17 +120,17 @@ impl AttachmentStore {
         let attachment = Self::get_attachment_impl(store_path, guid);
 
         if attachment.is_downloaded(preview) {
-            log::info!(target: target::ATTACHMENTS, "Attachment already downloaded: {}", attachment.guid);
+            log::debug!(target: target::ATTACHMENTS, "Attachment already downloaded: {}", attachment.guid);
             return Err(AttachmentStoreError::AttachmentAlreadyDownloaded.into());
         }
 
         let temporary_path = attachment.get_path_for_preview_scratch(preview, true);
         if std::fs::exists(&temporary_path).unwrap_or(false) {
-            log::info!(target: target::ATTACHMENTS, "Temporary file already exists: {}, assuming download is in progress", temporary_path.display());
+            log::warn!(target: target::ATTACHMENTS, "Temporary file already exists: {}, assuming download is in progress", temporary_path.display());
             return Err(AttachmentStoreError::DownloadAlreadyInProgress.into());
         }
 
-        log::info!(target: target::ATTACHMENTS, "Starting download for attachment: {}", attachment.guid);
+        log::debug!(target: target::ATTACHMENTS, "Starting download for attachment: {}", attachment.guid);
 
         let file = std::fs::File::create(&temporary_path)?;
         let mut writer = BufWriter::new(&file);
@@ -155,7 +155,7 @@ impl AttachmentStore {
             &attachment.get_path_for_preview_scratch(preview, false),
         )?;
 
-        log::info!(target: target::ATTACHMENTS, "Completed download for attachment: {}", attachment.guid);
+        log::debug!(target: target::ATTACHMENTS, "Completed download for attachment: {}", attachment.guid);
 
         // Send a signal to the daemon that the attachment has been downloaded.
         let event = DaemonEvent::AttachmentDownloaded(attachment.guid.clone());
@@ -240,7 +240,7 @@ impl AttachmentStore {
 
                                 log::debug!(target: target::ATTACHMENTS, "Queued download for attachment: {}", &guid);
                             } else {
-                                log::info!(target: target::ATTACHMENTS, "Attachment already downloaded: {}", guid);
+                                log::debug!(target: target::ATTACHMENTS, "Attachment already downloaded: {}", guid);
                             }
                         }
 
diff --git a/kordophoned/src/daemon/mod.rs b/kordophoned/src/daemon/mod.rs
index 3498e81..5f0daf3 100644
--- a/kordophoned/src/daemon/mod.rs
+++ b/kordophoned/src/daemon/mod.rs
@@ -374,7 +374,7 @@ impl Daemon {
             }
 
             Event::DownloadAttachment(attachment_id, preview, reply) => {
-                log::info!(target: target::ATTACHMENTS, "Download requested for attachment: {}, preview: {}", &attachment_id, preview);
+                log::debug!(target: target::ATTACHMENTS, "Download requested for attachment: {}, preview: {}", &attachment_id, preview);
 
                 self.attachment_store_sink
                     .as_ref()
@@ -390,7 +390,7 @@ impl Daemon {
             }
 
             Event::AttachmentDownloaded(attachment_id) => {
-                log::info!(target: target::ATTACHMENTS, "Daemon: attachment downloaded: {}, sending signal", attachment_id);
+                log::debug!(target: target::ATTACHMENTS, "Daemon: attachment downloaded: {}, sending signal", attachment_id);
 
                 // Send signal to the client that the attachment has been downloaded.
                 self.signal_sender
diff --git a/kordophoned/src/daemon/update_monitor.rs b/kordophoned/src/daemon/update_monitor.rs
index 329954b..c8be792 100644
--- a/kordophoned/src/daemon/update_monitor.rs
+++ b/kordophoned/src/daemon/update_monitor.rs
@@ -78,7 +78,7 @@ impl UpdateMonitor {
                 // imagent will post a conversation changed notification when we call getMessages.
                 if let Some(last_sync) = self.last_sync_times.get(&conversation.guid) {
                     if last_sync.elapsed() < Duration::from_secs(1) {
-                        log::info!(target: target::UPDATES, "Skipping sync for conversation id: {}. Last sync was {} seconds ago.", 
+                        log::warn!(target: target::UPDATES, "Skipping sync for conversation id: {}. Last sync was {} seconds ago.", 
                             conversation.guid, last_sync.elapsed().as_secs_f64());
                         return;
                     }
@@ -93,7 +93,7 @@ impl UpdateMonitor {
                 match (&last_message, &conversation.last_message) {
                     (Some(message), Some(conversation_message)) => {
                         if message.id == conversation_message.guid {
-                            log::info!(target: target::UPDATES, "Skipping sync for conversation id: {}. We already have this message.", &conversation.guid);
+                            log::warn!(target: target::UPDATES, "Skipping sync for conversation id: {}. We already have this message.", &conversation.guid);
                             return;
                         }
                     }