From c9928c06e60bd5bd9e4c69d826816a92ae4a6d1c Mon Sep 17 00:00:00 2001
From: Tobias Klika <github.8qp8l@alias.swcs.pro>
Date: Thu, 16 Apr 2026 15:07:11 +0200
Subject: [PATCH] fix errors when no config is passed; add health check
 endpoint;  add response caching

---
 Finch/ApplicationBuilderExtensions.cs |  4 +++-
 Finch/FinchBuilder.cs                 |  1 +
 Finch/Logging/FinchLoggingModule.cs   |  4 ++--
 Finch/Mvc/FinchMvcModule.cs           | 11 ++++++++++-
 4 files changed, 16 insertions(+), 4 deletions(-)

diff --git a/Finch/ApplicationBuilderExtensions.cs b/Finch/ApplicationBuilderExtensions.cs
index d307b302..ccb114ca 100644
--- a/Finch/ApplicationBuilderExtensions.cs
+++ b/Finch/ApplicationBuilderExtensions.cs
@@ -1,4 +1,5 @@
 using Microsoft.AspNetCore.Builder;
+using Microsoft.AspNetCore.Routing;
 
 namespace Finch;
 
@@ -7,6 +8,7 @@ public static class ApplicationBuilderExtensions
   public static IApplicationBuilder UseFinch(this IApplicationBuilder app)
   {
     app.UseMiddleware<FinchContextMiddleware>();
+    app.UseResponseCaching();
     app.UseOutputCache();
 
     if (app is WebApplication webApplication)
@@ -15,7 +17,7 @@ public static class ApplicationBuilderExtensions
       webApplication.MapControllers();
     }
 
-    FinchBuilder.Modules.Configure(app, null, app.ApplicationServices);
+    FinchBuilder.Modules.Configure(app, app as IEndpointRouteBuilder, app.ApplicationServices);
     return app;
   }
 }
diff --git a/Finch/FinchBuilder.cs b/Finch/FinchBuilder.cs
index 64e66bde..ee9c3b27 100644
--- a/Finch/FinchBuilder.cs
+++ b/Finch/FinchBuilder.cs
@@ -38,6 +38,7 @@ public class FinchBuilder
       startupOptions.AssemblyDiscoveryRules.Add(new FinchAssemblyDiscoveryRule());
       setupAction?.Invoke(startupOptions);
 
+      services.AddResponseCaching();
       services.AddControllers();
       services.AddOutputCache();
       mvcBuilder = services.AddRazorPages();
diff --git a/Finch/Logging/FinchLoggingModule.cs b/Finch/Logging/FinchLoggingModule.cs
index 4086c802..e03f6dd9 100644
--- a/Finch/Logging/FinchLoggingModule.cs
+++ b/Finch/Logging/FinchLoggingModule.cs
@@ -15,7 +15,7 @@ public class FinchLoggingModule : FinchModule
     {
       // get seq configuration
       IConfigurationSection seqConfig = configuration.GetSection("Finch:Seq");
-      FinchSeqOptions seqOptions = seqConfig.Get<FinchSeqOptions>();
+      FinchSeqOptions seqOptions = seqConfig.Get<FinchSeqOptions>() ?? new();
 
       // default level
       builder.SetMinimumLevel(seqOptions.MinimumLevel);
@@ -27,7 +27,7 @@ public class FinchLoggingModule : FinchModule
       }
 
       // add optional seq sink
-      if (seqConfig.Exists())
+      if (seqConfig.Exists() && seqOptions.ApiKey.HasValue())
       {
         builder.AddSeq(seqOptions.ServerUrl, seqOptions.ApiKey, seqOptions.Enrichers);
       }
diff --git a/Finch/Mvc/FinchMvcModule.cs b/Finch/Mvc/FinchMvcModule.cs
index 8ca9a33e..22a69026 100644
--- a/Finch/Mvc/FinchMvcModule.cs
+++ b/Finch/Mvc/FinchMvcModule.cs
@@ -1,4 +1,6 @@
-using Microsoft.Extensions.Configuration;
+using Microsoft.AspNetCore.Builder;
+using Microsoft.AspNetCore.Routing;
+using Microsoft.Extensions.Configuration;
 using Microsoft.Extensions.DependencyInjection;
 
 namespace Finch.Mvc;
@@ -8,5 +10,12 @@ public class FinchMvcModule : FinchModule
   public override void ConfigureServices(IServiceCollection services, IConfiguration configuration)
   {
     services.AddRouting(opts => opts.LowercaseUrls = true);
+    services.AddHealthChecks();
+  }
+
+
+  public override void Configure(IApplicationBuilder app, IEndpointRouteBuilder routes, IServiceProvider serviceProvider)
+  {
+    routes.MapHealthChecks("/health");
   }
 }
\ No newline at end of file