Also update IWireMockMiddlewareOptions when settings are updated via admin interface (#1166)

* Also update IWireMockMiddlewareOptions when settings are updated via admin interface

* ci

azure-pipelines-ci.yml

@@ -53,8 +53,13 @@ jobs:
     inputs:
       script: |
           dotnet-coverage collect "dotnet test ./test/WireMock.Net.Tests/WireMock.Net.Tests.csproj --configuration Debug --no-build --framework net8.0" -f xml -o "wiremock-coverage-1.xml"
+      displayName: 'Execute WireMock.Net.Tests with Coverage'
+
+  - task: CmdLine@2
+    inputs:
+      script: |
           dotnet-coverage collect "dotnet test ./test/WireMock.Net.Aspire.Tests/WireMock.Net.Aspire.Tests.csproj --configuration Debug --no-build" -f xml -o "wiremock-coverage-2.xml"
-    displayName: 'Execute Unit Tests with Coverage'
+    displayName: 'Execute WireMock.Net.Aspire.Tests with Coverage'
 
   - task: CmdLine@2
     displayName: 'Merge coverage files'

src/WireMock.Net/Owin/WireMockMiddlewareOptionsHelper.cs

@@ -1,5 +1,6 @@
 // Copyright © WireMock.Net
 
+using System;
 using Stef.Validation;
 using WireMock.Settings;
 
@@ -7,21 +8,31 @@ namespace WireMock.Owin;
 
 internal static class WireMockMiddlewareOptionsHelper
 {
-    public static IWireMockMiddlewareOptions InitFromSettings(WireMockServerSettings settings, IWireMockMiddlewareOptions? options = null)
+    public static IWireMockMiddlewareOptions InitFromSettings(
+        WireMockServerSettings settings,
+        IWireMockMiddlewareOptions? options = null,
+        Action<IWireMockMiddlewareOptions>? postConfigure = null
+    )
     {
         Guard.NotNull(settings);
 
         options ??= new WireMockMiddlewareOptions();
 
-        options.FileSystemHandler = settings.FileSystemHandler;
+        options.AllowBodyForAllHttpMethods = settings.AllowBodyForAllHttpMethods;
-        options.PreWireMockMiddlewareInit = settings.PreWireMockMiddlewareInit;
+        options.AllowOnlyDefinedHttpStatusCodeInResponse = settings.AllowOnlyDefinedHttpStatusCodeInResponse;
-        options.PostWireMockMiddlewareInit = settings.PostWireMockMiddlewareInit;
+        options.AllowPartialMapping = settings.AllowPartialMapping;
-        options.Logger = settings.Logger;
         options.DisableJsonBodyParsing = settings.DisableJsonBodyParsing;
-        options.HandleRequestsSynchronously = settings.HandleRequestsSynchronously;
+        options.DisableRequestBodyDecompressing = settings.DisableRequestBodyDecompressing;
-        options.SaveUnmatchedRequests = settings.SaveUnmatchedRequests;
         options.DoNotSaveDynamicResponseInLogEntry = settings.DoNotSaveDynamicResponseInLogEntry;
+        options.FileSystemHandler = settings.FileSystemHandler;
+        options.HandleRequestsSynchronously = settings.HandleRequestsSynchronously;
+        options.Logger = settings.Logger;
+        options.MaxRequestLogCount = settings.MaxRequestLogCount;
+        options.PostWireMockMiddlewareInit = settings.PostWireMockMiddlewareInit;
+        options.PreWireMockMiddlewareInit = settings.PreWireMockMiddlewareInit;
         options.QueryParameterMultipleValueSupport = settings.QueryParameterMultipleValueSupport;
+        options.RequestLogExpirationDuration = settings.RequestLogExpirationDuration;
+        options.SaveUnmatchedRequests = settings.SaveUnmatchedRequests;
 
         if (settings.CustomCertificateDefined)
         {
@@ -32,6 +43,8 @@ internal static class WireMockMiddlewareOptionsHelper
             options.X509CertificatePassword = settings.CertificateSettings.X509CertificatePassword;
         }
 
+        postConfigure?.Invoke(options);
+
         return options;
     }
 }

src/WireMock.Net/Server/WireMockServer.Admin.cs

@@ -18,6 +18,7 @@ using WireMock.Http;
 using WireMock.Logging;
 using WireMock.Matchers;
 using WireMock.Matchers.Request;
+using WireMock.Owin;
 using WireMock.RequestBuilders;
 using WireMock.ResponseProviders;
 using WireMock.Serialization;
@@ -321,29 +322,27 @@ public partial class WireMockServer
 
         InitSettings(_settings);
 
-        // _options
-        if (settings.GlobalProcessingDelay != null)
-        {
-            _options.RequestProcessingDelay = TimeSpan.FromMilliseconds(settings.GlobalProcessingDelay.Value);
-        }
-        _options.AllowBodyForAllHttpMethods = settings.AllowBodyForAllHttpMethods;
-        _options.AllowPartialMapping = settings.AllowPartialMapping;
-        _options.HandleRequestsSynchronously = settings.HandleRequestsSynchronously;
-        _options.MaxRequestLogCount = settings.MaxRequestLogCount;
-        _options.RequestLogExpirationDuration = settings.RequestLogExpirationDuration;
-
-        // _settings & _options
 #if USE_ASPNETCORE
         if (Enum.TryParse<CorsPolicyOptions>(settings.CorsPolicyOptions, true, out var corsPolicyOptions))
         {
             _settings.CorsPolicyOptions = corsPolicyOptions;
-            _options.CorsPolicyOptions = corsPolicyOptions;
         }
-
-        _options.ClientCertificateMode = _settings.ClientCertificateMode;
-        _options.AcceptAnyClientCertificate = _settings.AcceptAnyClientCertificate;
 #endif
 
+        WireMockMiddlewareOptionsHelper.InitFromSettings(_settings, _options, o =>
+        {
+            if (settings.GlobalProcessingDelay != null)
+            {
+                o.RequestProcessingDelay = TimeSpan.FromMilliseconds(settings.GlobalProcessingDelay.Value);
+            }
+
+#if USE_ASPNETCORE
+            o.CorsPolicyOptions = corsPolicyOptions;
+            o.ClientCertificateMode = _settings.ClientCertificateMode;
+            o.AcceptAnyClientCertificate = _settings.AcceptAnyClientCertificate;
+#endif
+        });
+
         return ResponseMessageBuilder.Create(200, "Settings updated");
     }
     #endregion Settings

src/WireMock.Net/Server/WireMockServer.cs

@@ -20,7 +20,6 @@ using WireMock.Exceptions;
 using WireMock.Handlers;
 using WireMock.Http;
 using WireMock.Logging;
-using WireMock.Matchers.Request;
 using WireMock.Models;
 using WireMock.Owin;
 using WireMock.RequestBuilders;
@@ -369,9 +368,10 @@ public partial class WireMockServer : IWireMockServer
             }
         }
 
-        WireMockMiddlewareOptionsHelper.InitFromSettings(settings, _options);
+        WireMockMiddlewareOptionsHelper.InitFromSettings(settings, _options, o =>
-
+        {
-        _options.LogEntries.CollectionChanged += LogEntries_CollectionChanged;
+            o.LogEntries.CollectionChanged += LogEntries_CollectionChanged;
+        });
 
         _matcherMapper = new MatcherMapper(_settings);
         _mappingConverter = new MappingConverter(_matcherMapper);
@@ -649,13 +649,11 @@ public partial class WireMockServer : IWireMockServer
     {
         if (settings.AllowBodyForAllHttpMethods == true)
         {
-            _options.AllowBodyForAllHttpMethods = _settings.AllowBodyForAllHttpMethods;
             _settings.Logger.Info("AllowBodyForAllHttpMethods is set to True");
         }
 
         if (settings.AllowOnlyDefinedHttpStatusCodeInResponse == true)
         {
-            _options.AllowOnlyDefinedHttpStatusCodeInResponse = _settings.AllowOnlyDefinedHttpStatusCodeInResponse;
             _settings.Logger.Info("AllowOnlyDefinedHttpStatusCodeInResponse is set to True");
         }