From ad065892fdbbeb00357f1609e158e76f8778db8f Mon Sep 17 00:00:00 2001
From: Kiran Challa <kichalla@microsoft.com>
Date: Mon, 2 May 2016 10:59:20 -0700
Subject: [PATCH] [Fixes #4575] Logging ArgumentOutOfRangeException

---
 .../Internal/MvcCoreLoggerExtensions.cs                     | 6 +++---
 .../Internal/MvcJsonLoggerExtensions.cs                     | 6 +++---
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/Microsoft.AspNetCore.Mvc.Core/Internal/MvcCoreLoggerExtensions.cs b/src/Microsoft.AspNetCore.Mvc.Core/Internal/MvcCoreLoggerExtensions.cs
index 7422e58e70..9246717ad9 100644
--- a/src/Microsoft.AspNetCore.Mvc.Core/Internal/MvcCoreLoggerExtensions.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Core/Internal/MvcCoreLoggerExtensions.cs
@@ -53,7 +53,7 @@ namespace Microsoft.AspNetCore.Mvc.Internal
         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 @@ namespace Microsoft.AspNetCore.Mvc.Internal
                 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 @@ namespace Microsoft.AspNetCore.Mvc.Internal
 
         public static void NoAcceptForNegotiation(this ILogger logger)
         {
-            _noAcceptForNegotiation(logger, null, null);
+            _noAcceptForNegotiation(logger, null);
         }
 
         public static void NoFormatterFromNegotiation(this ILogger logger, IList<MediaTypeSegmentWithQuality> acceptTypes)
diff --git a/src/Microsoft.AspNetCore.Mvc.Formatters.Json/Internal/MvcJsonLoggerExtensions.cs b/src/Microsoft.AspNetCore.Mvc.Formatters.Json/Internal/MvcJsonLoggerExtensions.cs
index df064c4bb6..8936571c26 100644
--- a/src/Microsoft.AspNetCore.Mvc.Formatters.Json/Internal/MvcJsonLoggerExtensions.cs
+++ b/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 @@ namespace Microsoft.AspNetCore.Mvc.Formatters.Json.Internal
 
         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)