From b2385709f8081c93eb30a1db25b7e5e6c91ce99e Mon Sep 17 00:00:00 2001 From: Joseph Doherty Date: Wed, 18 Mar 2026 01:08:44 -0400 Subject: [PATCH] fix: raise health report sender log level to INFO for observability Changed "Sent health report" from DEBUG to INFO and failure log from WARNING to ERROR so health report activity is visible in default logging. --- src/ScadaLink.HealthMonitoring/HealthReportSender.cs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/ScadaLink.HealthMonitoring/HealthReportSender.cs b/src/ScadaLink.HealthMonitoring/HealthReportSender.cs index 4eda01b..a2bf904 100644 --- a/src/ScadaLink.HealthMonitoring/HealthReportSender.cs +++ b/src/ScadaLink.HealthMonitoring/HealthReportSender.cs @@ -61,11 +61,11 @@ public class HealthReportSender : BackgroundService _transport.Send(reportWithSeq); - _logger.LogDebug("Sent health report #{Seq} for site {SiteId}", seq, _siteId); + _logger.LogInformation("Sent health report #{Seq} for site {SiteId}", seq, _siteId); } catch (Exception ex) { - _logger.LogWarning(ex, "Failed to send health report for site {SiteId}", _siteId); + _logger.LogError(ex, "Failed to send health report for site {SiteId}", _siteId); // Continue sending — don't let a single failure stop reporting } }