[Fixes #4575] Logging ArgumentOutOfRangeException

Author: kichalla

src/Microsoft.AspNetCore.Mvc.Core/Internal/MvcCoreLoggerExtensions.cs

@@ -53,7 +53,7 @@ internal static class MvcCoreLoggerExtensions
         private static readonly Action<ILogger, string, Exception> _noFormatter;
         private static readonly Action<ILogger, IOutputFormatter, string, Exception> _formatterSelected;
         private static readonly Action<ILogger, string, Exception> _skippedContentNegotiation;
-        private static readonly Action<ILogger, string, Exception> _noAcceptForNegotiation;
+        private static readonly Action<ILogger, Exception> _noAcceptForNegotiation;
         private static readonly Action<ILogger, IEnumerable<MediaTypeSegmentWithQuality>, Exception> _noFormatterFromNegotiation;
 
         private static readonly Action<ILogger, string, Exception> _redirectResultExecuting;
@@ -174,7 +174,7 @@ static MvcCoreLoggerExtensions()
                 3,
                 "Skipped content negotiation as content type '{ContentType}' is explicitly set for the response.");
 
-            _noAcceptForNegotiation = LoggerMessage.Define<string>(
+            _noAcceptForNegotiation = LoggerMessage.Define(
                 LogLevel.Debug,
                 4,
                 "No information found on request to perform content negotiation.");
@@ -369,7 +369,7 @@ public static void SkippedContentNegotiation(this ILogger logger, string content
 
         public static void NoAcceptForNegotiation(this ILogger logger)
         {
-            _noAcceptForNegotiation(logger, null, null);
+            _noAcceptForNegotiation(logger, null);
         }
 
         public static void NoFormatterFromNegotiation(this ILogger logger, IList<MediaTypeSegmentWithQuality> acceptTypes)

src/Microsoft.AspNetCore.Mvc.Formatters.Json/Internal/MvcJsonLoggerExtensions.cs

@@ -8,13 +8,13 @@ namespace Microsoft.AspNetCore.Mvc.Formatters.Json.Internal
 {
     internal static class MvcJsonLoggerExtensions
     {
-        private static readonly Action<ILogger, string, Exception> _jsonInputFormatterCrashed;
+        private static readonly Action<ILogger, Exception> _jsonInputFormatterCrashed;
 
         private static readonly Action<ILogger, string, Exception> _jsonResultExecuting;
 
         static MvcJsonLoggerExtensions()
         {
-            _jsonInputFormatterCrashed = LoggerMessage.Define<string>(
+            _jsonInputFormatterCrashed = LoggerMessage.Define(
                 LogLevel.Debug,
                 1,
                 "JSON input formatter threw an exception.");
@@ -27,7 +27,7 @@ static MvcJsonLoggerExtensions()
 
         public static void JsonInputException(this ILogger logger, Exception exception)
         {
-            _jsonInputFormatterCrashed(logger, exception.ToString(), exception);
+            _jsonInputFormatterCrashed(logger, exception);
         }
 
         public static void JsonResultExecuting(this ILogger logger, object value)