yv01p/Umbraco-CMS
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Avoid creating a new string for the log level every log message (#10934)

Browse Source
This commit is contained in:
Chad
2021-08-31 00:22:07 +12:00
committed by GitHub
parent 50dfb1fd92
commit b1d1a2fcf2
1 changed files with 7 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
src/Umbraco.Core/Logging/Serilog/Enrichers/Log4NetLevelMapperEnricher.cs
View File

@@ -11,7 +11,7 @@ namespace Umbraco.Core.Logging.Serilog.Enrichers
{
public void Enrich(LogEvent logEvent, ILogEventPropertyFactory propertyFactory)
{
var log4NetLevel = string.Empty;
string log4NetLevel;
switch (logEvent.Level)
{
@@ -28,21 +28,21 @@ namespace Umbraco.Core.Logging.Serilog.Enrichers
break;
case LogEventLevel.Information:
log4NetLevel = "INFO";
log4NetLevel = "INFO "; //Padded string so that all log levels are 5 chars long (needed to keep the txt log file lined up nicely)
break;
case LogEventLevel.Verbose:
log4NetLevel = "ALL";
log4NetLevel = "ALL "; //Padded string so that all log levels are 5 chars long (needed to keep the txt log file lined up nicely)
break;
case LogEventLevel.Warning:
log4NetLevel = "WARN";
log4NetLevel = "WARN "; //Padded string so that all log levels are 5 chars long (needed to keep the txt log file lined up nicely)
break;
default:
log4NetLevel = string.Empty;
break;
}
//Pad string so that all log levels are 5 chars long (needed to keep the txt log file lined up nicely)
log4NetLevel = log4NetLevel.PadRight(5);
logEvent.AddPropertyIfAbsent(propertyFactory.CreateProperty("Log4NetLevel", log4NetLevel));
}
}