From 1079db274ed97fc84f7a6ca756707e5fa30cd3c9 Mon Sep 17 00:00:00 2001 From: DanConwayDev Date: Fri, 9 Jan 2026 22:02:02 +0000 Subject: fix: downgrade negentropy timeout warning to debug level Negentropy diff timeouts are expected when relays don't support NIP-77. The relay responds with NOTICE 'unknown envelope label' and the timeout is hit before we recognize this is unsupported rather than a failure. Changes: - Downgrade from warn! to debug! in negentropy_sync_filter() (src/sync/relay_connection.rs:493) - Add comment explaining timeouts are common for non-NIP-77 relays - Update message to clarify timeout typically means no NIP-77 support The existing fallback mechanism (lines 505-509) properly handles this case and logs a one-time warning about falling back to REQ+EOSE. Discovered via production sync testing against wss://git.shakespeare.diy --- src/sync/relay_connection.rs | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'src/sync') diff --git a/src/sync/relay_connection.rs b/src/sync/relay_connection.rs index b86d298..e57e06c 100644 --- a/src/sync/relay_connection.rs +++ b/src/sync/relay_connection.rs @@ -486,11 +486,12 @@ impl RelayConnection { ); // Check for any failures + // Note: Timeouts are common for relays without NIP-77 support if !output.failed.is_empty() { - tracing::warn!( + tracing::debug!( relay = %self.url, failures = ?output.failed, - "Some relays failed during negentropy diff" + "Negentropy diff had failures (timeout usually means relay doesn't support NIP-77)" ); } -- cgit v1.2.3