Log Exception (#405)

examples/WireMock.Net.Service/WireMockLog4NetLogger.cs

@@ -1,4 +1,5 @@
-using log4net;
+using System;
+using log4net;
 using Newtonsoft.Json;
 using Wiremock.Net.Service;
 using WireMock.Admin.Requests;
@@ -30,6 +31,11 @@ namespace WireMock.Net.Service
             Log.ErrorFormat(formatString, args);
         }
 
+        public void Error(string message, Exception exception)
+        {
+            Log.Error(message, exception);
+        }
+
         public void DebugRequestResponse(LogEntryModel logEntryModel, bool isAdminRequest)
         {
             string message = JsonConvert.SerializeObject(logEntryModel, Formatting.Indented);

examples/WireMock.Net.StandAlone.NETCoreApp/WireMockLog4NetLogger.cs

@@ -1,7 +1,7 @@
-using log4net;
+using System;
+using log4net;
 using Newtonsoft.Json;
 using WireMock.Logging;
-using WireMock.Models.Requests;
 
 namespace WireMock.Net.StandAlone.NETCoreApp
 {
@@ -29,6 +29,11 @@ namespace WireMock.Net.StandAlone.NETCoreApp
             Log.ErrorFormat(formatString, args);
         }
 
+        public void Error(string message, Exception exception)
+        {
+            Log.Error(message, exception);
+        }
+
         public void DebugRequestResponse(LogEntryModel logEntryModel, bool isAdminRequest)
         {
             string message = JsonConvert.SerializeObject(logEntryModel, Formatting.Indented);

src/WireMock.Net.Abstractions/Logging/IWireMockLogger.cs

@@ -1,4 +1,5 @@
-using JetBrains.Annotations;
+using System;
+using JetBrains.Annotations;
 using WireMock.Admin.Requests;
 
 namespace WireMock.Logging
@@ -45,6 +46,14 @@ namespace WireMock.Logging
         [StringFormatMethod("formatString")]
         void Error([NotNull] string formatString, [NotNull] params object[] args);
 
+        /// <summary>
+        /// Writes the message at the Error level using the specified exception. 
+        /// </summary>
+        /// <param name="message">The message.</param>
+        /// <param name="exception">The exception.</param>
+        [PublicAPI]
+        void Error([NotNull] string message, [NotNull] Exception exception);
+
         /// <summary>
         /// Writes the LogEntryModel (LogRequestModel, LogResponseModel and more).
         /// </summary>

src/WireMock.Net/Logging/WireMockConsoleLogger.cs

@@ -36,12 +36,27 @@ namespace WireMock.Logging
             Console.WriteLine(Format("Warn", formatString, args));
         }
 
-        /// <see cref="IWireMockLogger.Error"/>
+        /// <see cref="IWireMockLogger.Error(string, object[])"/>
         public void Error(string formatString, params object[] args)
         {
             Console.WriteLine(Format("Error", formatString, args));
         }
 
+        /// <see cref="IWireMockLogger.Error(string, Exception)"/>
+        public void Error(string formatString, Exception exception)
+        {
+            Console.WriteLine(Format("Error", formatString, exception.Message));
+
+            if (exception is AggregateException ae)
+            {
+                ae.Handle(ex =>
+                {
+                    Console.WriteLine(Format("Error", "Exception {0}", exception.Message));
+                    return true;
+                });
+            }
+        }
+
         /// <see cref="IWireMockLogger.DebugRequestResponse"/>
         public void DebugRequestResponse(LogEntryModel logEntryModel, bool isAdminRequest)
         {

src/WireMock.Net/Logging/WireMockNullLogger.cs

@@ -1,4 +1,5 @@
-using WireMock.Admin.Requests;
+using System;
+using WireMock.Admin.Requests;
 
 namespace WireMock.Logging
 {
@@ -26,12 +27,18 @@ namespace WireMock.Logging
             // Log nothing
         }
 
-        /// <see cref="IWireMockLogger.Error"/>
+        /// <see cref="IWireMockLogger.Error(string, object[])"/>
         public void Error(string formatString, params object[] args)
         {
             // Log nothing
         }
 
+        /// <see cref="IWireMockLogger.Error(string, Exception)"/>
+        public void Error(string formatString, Exception exception)
+        {
+            // Log nothing
+        }
+
         /// <see cref="IWireMockLogger.DebugRequestResponse"/>
         public void DebugRequestResponse(LogEntryModel logEntryModel, bool isAdminRequest)
         {

src/WireMock.Net/Owin/GlobalExceptionMiddleware.cs

@@ -63,7 +63,7 @@ namespace WireMock.Owin
             }
             catch (Exception ex)
             {
-                _options.Logger.Error("HttpStatusCode set to 500 {0}", ex);
+                _options.Logger.Error("HttpStatusCode set to 500", ex);
                 await _responseMapper.MapAsync(ResponseMessageBuilder.Create(JsonConvert.SerializeObject(ex), 500), ctx.Response);
             }
         }

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

@@ -490,12 +490,12 @@ namespace WireMock.Server
             }
             catch (ArgumentException a)
             {
-                _settings.Logger.Error("HttpStatusCode set to 400 {0}", a);
+                _settings.Logger.Error("HttpStatusCode set to 400", a);
                 return ResponseMessageBuilder.Create(a.Message, 400);
             }
             catch (Exception e)
             {
-                _settings.Logger.Error("HttpStatusCode set to 500 {0}", e);
+                _settings.Logger.Error("HttpStatusCode set to 500", e);
                 return ResponseMessageBuilder.Create(e.ToString(), 500);
             }
         }