Merge pull request #32 from allegro/feature/error-handling-middleware…

…-use-service-provider feat: new `AddFluentErrorHandlingMiddleware` overload that allows to pass factories for logging methods
allegro · Jan 12, 2024 · 174e09c · 174e09c
2 parents 8f6b36f + 6dd6e7b
commit 174e09c
Show file tree

Hide file tree

Showing 3 changed files with 31 additions and 3 deletions.
diff --git a/...pNetCore/Allegro.Extensions.AspNetCore/ErrorHandling/ErrorHandlingMiddlewareExtensions.cs b/...pNetCore/Allegro.Extensions.AspNetCore/ErrorHandling/ErrorHandlingMiddlewareExtensions.cs
@@ -27,6 +27,28 @@ public static IServiceCollection AddFluentErrorHandlingMiddleware(
         Action<(string Message, Exception Exception)> logWarning,
         Action<ErrorHandlingConfigurationBuilder>? customErrorHandlerRegistration = null,
         IErrorSerializer? errorSerializer = null)
+    {
+        return services.AddFluentErrorHandlingMiddleware(
+            _ => logError,
+            _ => logWarning,
+            customErrorHandlerRegistration,
+            errorSerializer);
+    }
+
+    /// <summary>
+    /// Adds fluent error handling support to application
+    /// </summary>
+    /// <param name="services">Service collections</param>
+    /// <param name="logErrorFactory">Factory of delegate to log errors - enables to use any logger technology without Microsoft ILogger/<T/></param>
+    /// <param name="logWarningFactory">Factory of delegate to log warnings - enables to use any logger technology without Microsoft ILogger/<T/></param>
+    /// <param name="customErrorHandlerRegistration">Custom handler configurations</param>
+    /// <param name="errorSerializer">Enables to use other kind of error serialization than default System.Text.Json in Web mode</param>
+    public static IServiceCollection AddFluentErrorHandlingMiddleware(
+        this IServiceCollection services,
+        Func<IServiceProvider, Action<(string Message, Exception Exception)>> logErrorFactory,
+        Func<IServiceProvider, Action<(string Message, Exception Exception)>> logWarningFactory,
+        Action<ErrorHandlingConfigurationBuilder>? customErrorHandlerRegistration = null,
+        IErrorSerializer? errorSerializer = null)
     {
         var errorHandlingConfigurationBuilder = CreateErrorHandlingConfigurationBuilder(customErrorHandlerRegistration);
 
@@ -35,8 +57,8 @@ public static IServiceCollection AddFluentErrorHandlingMiddleware(
                 errorHandlingConfigurationBuilder.CustomErrorHandlerMap,
                 errorHandlingConfigurationBuilder.AdditionalInstrumentation,
                 errorSerializer ?? new SystemTextJsonWebErrorSerializer(),
-                logError,
-                logWarning));
+                logErrorFactory(provider),
+                logWarningFactory(provider)));
         return services;
     }
 

diff --git a/src/Allegro.Extensions.AspNetCore/CHANGELOG.md b/src/Allegro.Extensions.AspNetCore/CHANGELOG.md
@@ -5,6 +5,12 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/), and this project adheres
 to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.4.0] - 2024-01-11
+
+### Added
+
+* New `AddFluentErrorHandlingMiddleware` overload that allows to pass factories for logging methods.
+
 ## [1.3.0] - 2022-06-03
 
 ### Added

diff --git a/src/Allegro.Extensions.AspNetCore/version.xml b/src/Allegro.Extensions.AspNetCore/version.xml
@@ -1,5 +1,5 @@
 <Project>
   <PropertyGroup>
-    <VersionPrefix>1.3.0</VersionPrefix>
+    <VersionPrefix>1.4.0</VersionPrefix>
   </PropertyGroup>
 </Project>