Skip to content

Commit a33cdd4

Browse files
Reduce logging. Renew will log a warning on any failure, so don't need to warn again.
1 parent 47cc713 commit a33cdd4

File tree

1 file changed

+2
-10
lines changed

1 file changed

+2
-10
lines changed

src/zm_monitor_onvif.cpp

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -581,15 +581,7 @@ void ONVIF::WaitForMessage() {
581581
} // end foreach msg
582582
} // end scope for lock
583583

584-
// Log subscription timing before checking if renewal is needed
585-
log_subscription_timing("before_renewal_check");
586-
587-
// Renew subscription if needed
588-
if (IsRenewalNeeded()) {
589-
if (!Renew()) {
590-
Warning("ONVIF: Failed to renew subscription");
591-
}
592-
}
584+
if (IsRenewalNeeded()) Renew();
593585
} // end if SOAP OK/NOT OK
594586
#endif
595587
return;
@@ -947,7 +939,7 @@ bool ONVIF::Renew() {
947939
return true; // Not a fatal error
948940
} else {
949941
// Renewal failed - clean up the subscription to prevent leaks
950-
Debug(1, "ONVIF: Renewal failed, cleaning up subscription to prevent leak");
942+
Warning("ONVIF: Renewal failed, cleaning up subscription to prevent leak");
951943
cleanup_subscription();
952944
healthy = false;
953945
return false;

0 commit comments

Comments
 (0)