From ce4a19a20abe68cb3d677acdb498c2421b3c0666 Mon Sep 17 00:00:00 2001
From: Warren <warren@umbraco.com>
Date: Thu, 16 Aug 2018 15:24:45 +0100
Subject: [PATCH] Update OWIN logger with message template properties - mostly
 used for user X logged in (however we can't log those messages with
 properties) :(

---
 src/Umbraco.Core/Logging/OwinLogger.cs | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/src/Umbraco.Core/Logging/OwinLogger.cs b/src/Umbraco.Core/Logging/OwinLogger.cs
index a6ae0c332a..692065d150 100644
--- a/src/Umbraco.Core/Logging/OwinLogger.cs
+++ b/src/Umbraco.Core/Logging/OwinLogger.cs
@@ -26,34 +26,34 @@ namespace Umbraco.Core.Logging
             switch (eventType)
             {
                 case TraceEventType.Critical:
-                    _logger.Error(_type.Value, string.Format("Event Id: {0}, state: {1}", eventId, state), exception ?? new Exception("Critical error"));
+                    _logger.Error(_type.Value, "[{EventType}] Event Id: {EventId}, State: {State}", exception ?? new Exception("Critical error"), eventType, eventId, state);
                     return true;
                 case TraceEventType.Error:
-                    _logger.Error(_type.Value, string.Format("Event Id: {0}, state: {1}", eventId, state), exception ?? new Exception("Error"));
+                    _logger.Error(_type.Value, "[{EventType}] Event Id: {EventId}, State: {State}", exception ?? new Exception("Error"), eventType, eventId, state);
                     return true;
                 case TraceEventType.Warning:
-                    _logger.Warn(_type.Value, string.Format("Event Id: {0}, state: {1}", eventId, state));
+                    _logger.Warn(_type.Value, "[{EventType}] Event Id: {EventId}, State: {State}", eventType, eventId, state);
                     return true;
                 case TraceEventType.Information:
-                    _logger.Info(_type.Value, string.Format("Event Id: {0}, state: {1}", eventId, state));
+                    _logger.Info(_type.Value, "[{EventType}] Event Id: {EventId}, State: {State}", eventType, eventId, state);
                     return true;
                 case TraceEventType.Verbose:
-                    _logger.Debug(_type.Value, string.Format("Event Id: {0}, state: {1}", eventId, state));
+                    _logger.Debug(_type.Value, "[{EventType}] Event Id: {EventId}, State: {State}", eventType, eventId, state);
                     return true;
                 case TraceEventType.Start:
-                    _logger.Debug(_type.Value, string.Format("Event Id: {0}, state: {1}", eventId, state));
+                    _logger.Debug(_type.Value, "[{EventType}] Event Id: {EventId}, State: {State}", eventType, eventId, state);
                     return true;
                 case TraceEventType.Stop:
-                    _logger.Debug(_type.Value, string.Format("Event Id: {0}, state: {1}", eventId, state));
+                    _logger.Debug(_type.Value, "[{EventType}] Event Id: {EventId}, State: {State}", eventType, eventId, state);
                     return true;
                 case TraceEventType.Suspend:
-                    _logger.Debug(_type.Value, string.Format("Event Id: {0}, state: {1}", eventId, state));
+                    _logger.Debug(_type.Value, "[{EventType}] Event Id: {EventId}, State: {State}", eventType, eventId, state);
                     return true;
                 case TraceEventType.Resume:
-                    _logger.Debug(_type.Value, string.Format("Event Id: {0}, state: {1}", eventId, state));
+                    _logger.Debug(_type.Value, "[{EventType}] Event Id: {EventId}, State: {State}", eventType, eventId, state);
                     return true;
                 case TraceEventType.Transfer:
-                    _logger.Debug(_type.Value, string.Format("Event Id: {0}, state: {1}", eventId, state));
+                    _logger.Debug(_type.Value, "[{EventType}] Event Id: {EventId}, State: {State}", eventType, eventId, state);
                     return true;
                 default:
                     throw new ArgumentOutOfRangeException("eventType");