From b3bb103da5ec411fdfae556bd29b7aa4cd2011ea Mon Sep 17 00:00:00 2001
From: Mark Ridgwell <credfeto@users.noreply.github.com>
Date: Tue, 23 Jun 2026 10:24:05 +0000
Subject: [PATCH 1/6] [Dependencies] Updating
 Credfeto.Docker.HealthCheck.Http.Client (Docker healthcheck http client) to
 0.0.72.928

---
 CHANGELOG.md                 | 2 ++
 src/BuildBot/BuildBot.csproj | 2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 6aceae1c..eff8f568 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -44,6 +44,7 @@ Please ADD ALL Changes to the UNRELEASED SECTION and not a specific release
 - Dependencies - Updated xunit.v3 to 3.2.2
 - Dependencies - Updated Serilog to 4.3.1
 - Dependencies - Updated Credfeto.Docker.HealthCheck.Http.Client to 0.0.61.659
+- Dependencies - Updated FunFair.Test.Common to 6.2.17.1982
 - Dependencies - Updated Figgle to 0.6.6
 - Dependencies - Updated AWSSDK to 4.0.8.11
 - Dependencies - Updated Discord.Net to 3.19.1
@@ -67,6 +68,7 @@ Please ADD ALL Changes to the UNRELEASED SECTION and not a specific release
 - Dependencies - Updated FunFair.Test.Common to 6.3.1.2342
 - Dependencies - Updated FunFair.CodeAnalysis to 7.2.2.2066
 - Dependencies - Updated Meziantou.Analyzer to 3.0.115
+- Dependencies - Updated Credfeto.Docker.HealthCheck.Http.Client to 0.0.72.928
 ### Removed
 ### Deployment Changes
 <!--
diff --git a/src/BuildBot/BuildBot.csproj b/src/BuildBot/BuildBot.csproj
index 443eab6b..81b6b509 100644
--- a/src/BuildBot/BuildBot.csproj
+++ b/src/BuildBot/BuildBot.csproj
@@ -81,7 +81,7 @@
     <ProjectReference Include="..\BuildBot.Watchtower\BuildBot.Watchtower.csproj" />
   </ItemGroup>
   <ItemGroup>
-    <PackageReference Include="Credfeto.Docker.HealthCheck.Http.Client" Version="0.0.61.659" />
+    <PackageReference Include="Credfeto.Docker.HealthCheck.Http.Client" Version="0.0.72.928" />
     <PackageReference Include="Microsoft.Extensions.Hosting.Systemd" Version="10.0.9" />
     <PackageReference Include="Microsoft.Extensions.Hosting.WindowsServices" Version="10.0.9" />
     <PackageReference Include="Serilog" Version="4.3.1" />

From 25e54592eb70671dfee159bfed6d185e7390c2b8 Mon Sep 17 00:00:00 2001
From: Mark Ridgwell <273118822+dnyw4l3n13@users.noreply.github.com>
Date: Tue, 23 Jun 2026 15:10:12 +0000
Subject: [PATCH 2/6] fix: pass ILogger to HealthCheckClient.ExecuteAsync after
 API change in 0.0.72.928

Credfeto.Docker.HealthCheck.Http.Client 0.0.72.928 added a required ILogger
parameter to ExecuteAsync. Pass NullLogger.Instance since the health check
runs before the host and DI container are initialised.

Prompt: Work on pull request #383 in funfair-tech/BuildBot.
---
 src/BuildBot/Program.cs | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/BuildBot/Program.cs b/src/BuildBot/Program.cs
index 91e478a6..5e39adf2 100644
--- a/src/BuildBot/Program.cs
+++ b/src/BuildBot/Program.cs
@@ -5,6 +5,7 @@
 using BuildBot.Helpers;
 using Credfeto.Docker.HealthCheck.Http.Client;
 using Microsoft.AspNetCore.Builder;
+using Microsoft.Extensions.Logging.Abstractions;
 
 namespace BuildBot;
 
@@ -20,7 +21,11 @@ public static class Program
     public static async Task<int> Main(string[] args)
     {
         return HealthCheckClient.IsHealthCheck(args: args, out string? checkUrl)
-            ? await HealthCheckClient.ExecuteAsync(targetUrl: checkUrl, cancellationToken: CancellationToken.None)
+            ? await HealthCheckClient.ExecuteAsync(
+                targetUrl: checkUrl,
+                logger: NullLogger.Instance,
+                cancellationToken: CancellationToken.None
+            )
             : await RunServerAsync(args);
     }
 

From 8161d9f94276850a524b5f02ee85ae5c7503a397 Mon Sep 17 00:00:00 2001
From: Mark Ridgwell <273118822+dnyw4l3n13@users.noreply.github.com>
Date: Tue, 23 Jun 2026 15:11:58 +0000
Subject: [PATCH 3/6] chore: update changelog for ILogger fix in
 HealthCheckClient.ExecuteAsync

Prompt: Work on pull request #383 in funfair-tech/BuildBot.
---
 CHANGELOG.md | 1 +
 1 file changed, 1 insertion(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index eff8f568..ad6fb7a8 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -19,6 +19,7 @@ Please ADD ALL Changes to the UNRELEASED SECTION and not a specific release
 - Suppress known Scriban 6.2.0 vulnerabilities pending upgrade
 - SnsMessage: Token property was never populated from the constructor argument
 - Added null guards for botMessageChannel and botReleaseMessageChannel parameters in BotService constructor
+- Pass ILogger to HealthCheckClient.ExecuteAsync to match new API in Credfeto.Docker.HealthCheck.Http.Client 0.0.72.928
 ### Changed
 - Dependencies - Updated NSubstitute.Analyzers.CSharp to 1.0.17
 - Switched to use minimal APIs

From d95babd8eb6651a292146985ea6e57923eaf508a Mon Sep 17 00:00:00 2001
From: Mark Ridgwell <273118822+dnyw4l3n13@users.noreply.github.com>
Date: Tue, 23 Jun 2026 15:22:42 +0000
Subject: [PATCH 4/6] fix: use console logger for health check to surface
 HealthCheckFailed errors

NullLogger.Instance was silently discarding the HealthCheckFailed(uri, exception)
Error-level log that ExecuteAsync emits on connection failures, TLS errors, and
DNS failures. The ILogger parameter was added to 0.0.72.928 specifically to
expose this diagnostic; use a minimal console LoggerFactory so operators can
see why a health check failed.

Prompt: Work on pull request #383 in funfair-tech/BuildBot.
---
 src/BuildBot/Program.cs | 17 +++++++++++------
 1 file changed, 11 insertions(+), 6 deletions(-)

diff --git a/src/BuildBot/Program.cs b/src/BuildBot/Program.cs
index 5e39adf2..c0371e9f 100644
--- a/src/BuildBot/Program.cs
+++ b/src/BuildBot/Program.cs
@@ -5,7 +5,7 @@
 using BuildBot.Helpers;
 using Credfeto.Docker.HealthCheck.Http.Client;
 using Microsoft.AspNetCore.Builder;
-using Microsoft.Extensions.Logging.Abstractions;
+using Microsoft.Extensions.Logging;
 
 namespace BuildBot;
 
@@ -20,13 +20,18 @@ public static class Program
     )]
     public static async Task<int> Main(string[] args)
     {
-        return HealthCheckClient.IsHealthCheck(args: args, out string? checkUrl)
-            ? await HealthCheckClient.ExecuteAsync(
+        if (HealthCheckClient.IsHealthCheck(args: args, out string? checkUrl))
+        {
+            using ILoggerFactory loggerFactory = LoggerFactory.Create(static builder => builder.AddConsole());
+
+            return await HealthCheckClient.ExecuteAsync(
                 targetUrl: checkUrl,
-                logger: NullLogger.Instance,
+                logger: loggerFactory.CreateLogger(nameof(Program)),
                 cancellationToken: CancellationToken.None
-            )
-            : await RunServerAsync(args);
+            );
+        }
+
+        return await RunServerAsync(args);
     }
 
     private static async ValueTask<int> RunServerAsync(string[] args)

From 52c2cc1bddbe6d83508d83d8bcfe793183a9a028 Mon Sep 17 00:00:00 2001
From: Mark Ridgwell <273118822+dnyw4l3n13@users.noreply.github.com>
Date: Wed, 24 Jun 2026 22:27:08 +0000
Subject: [PATCH 5/6] chore: remove stale
 Credfeto.Docker.HealthCheck.Http.Client 0.0.61.659 changelog entry

The [Unreleased] section contained both the old (0.0.61.659) and new
(0.0.72.928) versions of Credfeto.Docker.HealthCheck.Http.Client.
Since 0.0.61.659 was never shipped in a release, only the latest
version should appear in [Unreleased].
---
 CHANGELOG.md | 1 -
 1 file changed, 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index ad6fb7a8..3416d9d1 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -44,7 +44,6 @@ Please ADD ALL Changes to the UNRELEASED SECTION and not a specific release
 - Dependencies - Updated xunit.analyzers to 1.27.0
 - Dependencies - Updated xunit.v3 to 3.2.2
 - Dependencies - Updated Serilog to 4.3.1
-- Dependencies - Updated Credfeto.Docker.HealthCheck.Http.Client to 0.0.61.659
 - Dependencies - Updated FunFair.Test.Common to 6.2.17.1982
 - Dependencies - Updated Figgle to 0.6.6
 - Dependencies - Updated AWSSDK to 4.0.8.11

From 5fe4cd783a7ec9c4f536863a87220a81c335a197 Mon Sep 17 00:00:00 2001
From: Mark Ridgwell <273118822+dnyw4l3n13@users.noreply.github.com>
Date: Wed, 24 Jun 2026 22:27:35 +0000
Subject: [PATCH 6/6] chore: remove superseded FunFair.Test.Common 6.2.17.1982
 changelog entry

The [Unreleased] section contained FunFair.Test.Common 6.2.17.1982
alongside the newer 6.3.1.2342 in the same block. Since 6.2.17.1982
was never shipped in a release, the intermediate version entry adds
noise to the changelog without value.
---
 CHANGELOG.md | 1 -
 1 file changed, 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 3416d9d1..67180cea 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -44,7 +44,6 @@ Please ADD ALL Changes to the UNRELEASED SECTION and not a specific release
 - Dependencies - Updated xunit.analyzers to 1.27.0
 - Dependencies - Updated xunit.v3 to 3.2.2
 - Dependencies - Updated Serilog to 4.3.1
-- Dependencies - Updated FunFair.Test.Common to 6.2.17.1982
 - Dependencies - Updated Figgle to 0.6.6
 - Dependencies - Updated AWSSDK to 4.0.8.11
 - Dependencies - Updated Discord.Net to 3.19.1