From 847acdecb9c28a5307123b9ee685b769a598cfc1 Mon Sep 17 00:00:00 2001
From: DanConwayDev <DanConwayDev@protonmail.com>
Date: Tue, 27 Jan 2026 21:40:46 +0000
Subject: fix: distinguish 0 OIDs fetched from successful fetch in logging

When fetch_oids returns Ok(vec![]) (all requested OIDs missing from
remote), the log message now says 'Fetch returned no OIDs (not available
on remote)' instead of the misleading 'Fetch succeeded' with oids_fetched=0.
---
 src/purgatory/sync/functions.rs | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/purgatory/sync/functions.rs b/src/purgatory/sync/functions.rs
index 65d29af..2b7e71f 100644
--- a/src/purgatory/sync/functions.rs
+++ b/src/purgatory/sync/functions.rs
@@ -369,7 +369,7 @@ pub async fn sync_identifier_from_url<C: SyncContext + ?Sized>(
     throttle_manager.complete_request(&domain);
 
     let oids_fetched = match fetch_result {
-        Ok(fetched) => {
+        Ok(fetched) if !fetched.is_empty() => {
             debug!(
                 identifier = %identifier,
                 url = %url,
@@ -378,6 +378,14 @@ pub async fn sync_identifier_from_url<C: SyncContext + ?Sized>(
             );
             fetched.len()
         }
+        Ok(_) => {
+            debug!(
+                identifier = %identifier,
+                url = %url,
+                "Fetch returned no OIDs (not available on remote)"
+            );
+            0
+        }
         Err(e) => {
             debug!(
                 identifier = %identifier,
-- 
cgit v1.2.3