From a33cdd4444aa5348dfa10ffca774db058be8bef5 Mon Sep 17 00:00:00 2001 From: Isaac Connor Date: Sat, 17 Jan 2026 08:44:55 -0500 Subject: [PATCH] Reduce logging. Renew will log a warning on any failure, so don't need to warn again. --- src/zm_monitor_onvif.cpp | 12 ++---------- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/src/zm_monitor_onvif.cpp b/src/zm_monitor_onvif.cpp index 9e299ab8a..3ea3363c6 100644 --- a/src/zm_monitor_onvif.cpp +++ b/src/zm_monitor_onvif.cpp @@ -581,15 +581,7 @@ void ONVIF::WaitForMessage() { } // end foreach msg } // end scope for lock - // Log subscription timing before checking if renewal is needed - log_subscription_timing("before_renewal_check"); - - // Renew subscription if needed - if (IsRenewalNeeded()) { - if (!Renew()) { - Warning("ONVIF: Failed to renew subscription"); - } - } + if (IsRenewalNeeded()) Renew(); } // end if SOAP OK/NOT OK #endif return; @@ -947,7 +939,7 @@ bool ONVIF::Renew() { return true; // Not a fatal error } else { // Renewal failed - clean up the subscription to prevent leaks - Debug(1, "ONVIF: Renewal failed, cleaning up subscription to prevent leak"); + Warning("ONVIF: Renewal failed, cleaning up subscription to prevent leak"); cleanup_subscription(); healthy = false; return false;