From 161061dbbc062a77a971be883ccea08756e52f51 Mon Sep 17 00:00:00 2001 From: Nikolaj Date: Mon, 14 Sep 2020 14:10:19 +0200 Subject: [PATCH] Remove exception logging and finish LogWarning --- src/Umbraco.Core/Composing/TypeFinder.cs | 6 +- src/Umbraco.Core/Logging/ConsoleLogger.cs | 38 +++-------- .../Logging/DebugDiagnosticsLogger.cs | 38 ++--------- src/Umbraco.Core/Logging/DisposableTimer.cs | 2 +- src/Umbraco.Core/Logging/ILogger.cs | 64 ++----------------- src/Umbraco.Core/Logging/LoggerExtensions.cs | 27 +++----- src/Umbraco.Core/Logging/NullLogger.cs | 28 ++------ src/Umbraco.Core/Logging/ProfilingLogger.cs | 34 +++------- .../LuceneIndexDiagnostics.cs | 4 +- .../UmbracoExamineIndex.cs | 2 +- .../Compose/NotificationsComponent.cs | 4 +- .../Logging/Serilog/SerilogLogger.cs | 49 +++----------- .../Media/UploadAutoFillProperties.cs | 2 +- .../V_8_0_0/PropertyEditorsMigrationBase.cs | 2 +- .../Models/Mapping/PropertyTypeGroupMapper.cs | 3 +- .../Models/PathValidationExtensions.cs | 2 +- .../Models/PropertyTagsExtensions.cs | 2 +- .../Persistence/Factories/DataTypeFactory.cs | 4 +- .../RichTextEditorPastedImages.cs | 2 +- .../Runtime/MainDomSemaphoreLock.cs | 2 +- .../Controllers/DictionaryController.cs | 4 +- .../Controllers/RelationTypeController.cs | 4 +- .../ImageCropperTemplateExtensions.cs | 2 +- .../Install/InstallController.cs | 2 +- .../Middleware/UmbracoRequestMiddleware.cs | 2 +- src/Umbraco.Web/ImageProcessorLogger.cs | 4 +- src/Umbraco.Web/Logging/OwinLogger.cs | 4 +- .../AuthenticationOptionsExtensions.cs | 2 +- src/Umbraco.Web/UmbracoHelper.cs | 2 +- .../Filters/AngularAntiForgeryHelper.cs | 2 +- .../WebApi/UnhandledExceptionLogger.cs | 2 +- 31 files changed, 84 insertions(+), 261 deletions(-) diff --git a/src/Umbraco.Core/Composing/TypeFinder.cs b/src/Umbraco.Core/Composing/TypeFinder.cs index efd3a6cb23..0d4faf261b 100644 --- a/src/Umbraco.Core/Composing/TypeFinder.cs +++ b/src/Umbraco.Core/Composing/TypeFinder.cs @@ -297,7 +297,7 @@ namespace Umbraco.Core.Composing } catch (TypeLoadException ex) { - _logger.LogError(typeof(TypeFinder), ex, "Could not query types on {Assembly} assembly, this is most likely due to this assembly not being compatible with the current Umbraco version", assembly); + _logger.LogError(ex, "Could not query types on {Assembly} assembly, this is most likely due to this assembly not being compatible with the current Umbraco version", assembly); continue; } @@ -372,7 +372,7 @@ namespace Umbraco.Core.Composing } catch (TypeLoadException ex) { - _logger.LogError(typeof(TypeFinder), ex, "Could not query types on {Assembly} assembly, this is most likely due to this assembly not being compatible with the current Umbraco version", assembly); + _logger.LogError(ex, "Could not query types on {Assembly} assembly, this is most likely due to this assembly not being compatible with the current Umbraco version", assembly); continue; } @@ -442,7 +442,7 @@ namespace Umbraco.Core.Composing if (_notifiedLoadExceptionAssemblies.Contains(a.FullName) == false) { _notifiedLoadExceptionAssemblies.Add(a.FullName); - _logger.LogWarning(typeof (TypeFinder), ex, "Could not load all types from {TypeName}.", a.GetName().Name); + _logger.LogWarning(ex, "Could not load all types from {TypeName}.", a.GetName().Name); } } return rex.Types.WhereNotNull().ToArray(); diff --git a/src/Umbraco.Core/Logging/ConsoleLogger.cs b/src/Umbraco.Core/Logging/ConsoleLogger.cs index 26f119d0af..2e78b798b7 100644 --- a/src/Umbraco.Core/Logging/ConsoleLogger.cs +++ b/src/Umbraco.Core/Logging/ConsoleLogger.cs @@ -31,53 +31,31 @@ namespace Umbraco.Core.Logging Console.WriteLine("FATAL {0} - {1}", typeof(T).Name, _messageTemplates.Render(messageTemplate, propertyValues)); } - public void LogError(Type reporting, Exception exception, string message) - { - Console.WriteLine("ERROR {0} - {1}", reporting.Name, message); - Console.WriteLine(exception); - } - public void LogError(Type reporting, Exception exception) { Console.WriteLine("ERROR {0}", reporting.Name); Console.WriteLine(exception); } - public void LogError(Type reporting, string message) + public void LogError(Exception exception, string messageTemplate, params object[] propertyValues) { - Console.WriteLine("ERROR {0} - {1}", reporting.Name, message); - } - - public void LogError(Type reporting, Exception exception, string messageTemplate, params object[] propertyValues) - { - Console.WriteLine("ERROR {0} - {1}", reporting.Name, _messageTemplates.Render(messageTemplate, propertyValues)); + Console.WriteLine("ERROR {0} - {1}", typeof(T).Name, _messageTemplates.Render(messageTemplate, propertyValues)); Console.WriteLine(exception); } - public void LogError(Type reporting, string messageTemplate, params object[] propertyValues) + public void LogError(string messageTemplate, params object[] propertyValues) { - Console.WriteLine("ERROR {0} - {1}", reporting.Name, _messageTemplates.Render(messageTemplate, propertyValues)); + Console.WriteLine("ERROR {0} - {1}", typeof(T).Name, _messageTemplates.Render(messageTemplate, propertyValues)); } - public void LogWarning(Type reporting, string message) + public void LogWarning(string message, params object[] propertyValues) { - Console.WriteLine("WARN {0} - {1}", reporting.Name, message); + Console.WriteLine("WARN {0} - {1}", typeof(T).Name, _messageTemplates.Render(message, propertyValues)); } - public void LogWarning(Type reporting, string message, params object[] propertyValues) + public void LogWarning(Exception exception, string message, params object[] propertyValues) { - Console.WriteLine("WARN {0} - {1}", reporting.Name, _messageTemplates.Render(message, propertyValues)); - } - - public void LogWarning(Type reporting, Exception exception, string message) - { - Console.WriteLine("WARN {0} - {1}", reporting.Name, message); - Console.WriteLine(exception); - } - - public void LogWarning(Type reporting, Exception exception, string message, params object[] propertyValues) - { - Console.WriteLine("WARN {0} - {1}", reporting.Name, _messageTemplates.Render(message, propertyValues)); + Console.WriteLine("WARN {0} - {1}", typeof(T).Name, _messageTemplates.Render(message, propertyValues)); Console.WriteLine(exception); } diff --git a/src/Umbraco.Core/Logging/DebugDiagnosticsLogger.cs b/src/Umbraco.Core/Logging/DebugDiagnosticsLogger.cs index 9fe37dd40f..7dc880c4c5 100644 --- a/src/Umbraco.Core/Logging/DebugDiagnosticsLogger.cs +++ b/src/Umbraco.Core/Logging/DebugDiagnosticsLogger.cs @@ -35,12 +35,6 @@ namespace Umbraco.Core.Logging System.Diagnostics.Debug.WriteLine(messageTemplate, propertyValues); } - /// - public void LogError(Type reporting, Exception exception, string message) - { - System.Diagnostics.Debug.WriteLine(message + Environment.NewLine + exception, reporting.FullName); - } - /// public void LogError(Type reporting, Exception exception) { @@ -48,45 +42,27 @@ namespace Umbraco.Core.Logging } /// - public void LogError(Type reporting, string message) + public void LogError(Exception exception, string messageTemplate, params object[] propertyValues) { - System.Diagnostics.Debug.WriteLine(message); + System.Diagnostics.Debug.WriteLine(_messageTemplates.Render(messageTemplate, propertyValues) + Environment.NewLine + exception, typeof(T).FullName); } /// - public void LogError(Type reporting, Exception exception, string messageTemplate, params object[] propertyValues) - { - System.Diagnostics.Debug.WriteLine(_messageTemplates.Render(messageTemplate, propertyValues) + Environment.NewLine + exception, reporting.FullName); - } - - /// - public void LogError(Type reporting, string messageTemplate, params object[] propertyValues) + public void LogError(string messageTemplate, params object[] propertyValues) { System.Diagnostics.Debug.WriteLine(messageTemplate, propertyValues); } /// - public void LogWarning(Type reporting, string message) + public void LogWarning(string message, params object[] propertyValues) { - System.Diagnostics.Debug.WriteLine(message, reporting.FullName); + System.Diagnostics.Debug.WriteLine(_messageTemplates.Render(message, propertyValues), typeof(T).FullName); } /// - public void LogWarning(Type reporting, string message, params object[] propertyValues) + public void LogWarning(Exception exception, string message, params object[] propertyValues) { - System.Diagnostics.Debug.WriteLine(_messageTemplates.Render(message, propertyValues), reporting.FullName); - } - - /// - public void LogWarning(Type reporting, Exception exception, string message) - { - System.Diagnostics.Debug.WriteLine(message + Environment.NewLine + exception, reporting.FullName); - } - - /// - public void LogWarning(Type reporting, Exception exception, string message, params object[] propertyValues) - { - System.Diagnostics.Debug.WriteLine(_messageTemplates.Render(message + Environment.NewLine + exception, propertyValues), reporting.FullName); + System.Diagnostics.Debug.WriteLine(_messageTemplates.Render(message + Environment.NewLine + exception, propertyValues), typeof(T).FullName); } /// diff --git a/src/Umbraco.Core/Logging/DisposableTimer.cs b/src/Umbraco.Core/Logging/DisposableTimer.cs index 9ae1dd8e71..bd133a5fd2 100644 --- a/src/Umbraco.Core/Logging/DisposableTimer.cs +++ b/src/Umbraco.Core/Logging/DisposableTimer.cs @@ -84,7 +84,7 @@ namespace Umbraco.Core.Logging { if (_failed) { - _logger.LogError(_loggerType, _failException, "{FailMessage} ({Duration}ms) [Timing {TimingId}]", _failMessage, Stopwatch.ElapsedMilliseconds, _timingId); + _logger.LogError(_failException, "{FailMessage} ({Duration}ms) [Timing {TimingId}]", _failMessage, Stopwatch.ElapsedMilliseconds, _timingId); } else switch (_level) { diff --git a/src/Umbraco.Core/Logging/ILogger.cs b/src/Umbraco.Core/Logging/ILogger.cs index dfaa42f1aa..db58169ce3 100644 --- a/src/Umbraco.Core/Logging/ILogger.cs +++ b/src/Umbraco.Core/Logging/ILogger.cs @@ -3,7 +3,7 @@ namespace Umbraco.Core.Logging { - public interface ILogger : ILogger{} + public interface ILogger : ILogger { } /// /// Defines the logging service. @@ -24,14 +24,6 @@ namespace Umbraco.Core.Logging /// The level. bool IsEnabled(Type reporting, LogLevel level); - /// - /// Logs a fatal exception. - /// - /// The reporting type. - /// An exception. - /// The message string is unspecified and is implementation-specific. - void Fatal(Type reporting, Exception exception); - /// /// Logs a fatal message with an exception. /// @@ -50,74 +42,32 @@ namespace Umbraco.Core.Logging /// /// Logs an error message with an exception. /// - /// The reporting type. /// An exception. - /// A message. - void LogError(Type reporting, Exception exception, string message); - - /// - /// Logs an error exception. - /// - /// The reporting type. - /// An exception. - /// The message string is unspecified and is implementation-specific. - void LogError(Type reporting, Exception exception); + /// A message template. + /// Property values. + void LogError(Exception exception, string messageTemplate, params object[] propertyValues); /// /// Logs an error message. /// - /// The reporting type. - /// A message. - void LogError(Type reporting, string message); - - /// - /// Logs an error message with an exception. - /// - /// The reporting type. - /// An exception. /// A message template. /// Property values. - void LogError(Type reporting, Exception exception, string messageTemplate, params object[] propertyValues); - - /// - /// Logs an error message. - /// - /// The reporting type. - /// A message template. - /// Property values. - void LogError(Type reporting, string messageTemplate, params object[] propertyValues); + void LogError(string messageTemplate, params object[] propertyValues); /// /// Logs a warning message. /// - /// The reporting type. - /// A message. - void LogWarning(Type reporting, string message); - - /// - /// Logs a warning message. - /// - /// The reporting type. /// A message template. /// Property values. - void LogWarning(Type reporting, string messageTemplate, params object[] propertyValues); + void LogWarning(string messageTemplate, params object[] propertyValues); /// /// Logs a warning message with an exception. /// - /// The reporting type. - /// An exception. - /// A message. - void LogWarning(Type reporting, Exception exception, string message); - - /// - /// Logs a warning message with an exception. - /// - /// The reporting type. /// An exception. /// A message template. /// Property values. - void LogWarning(Type reporting, Exception exception, string messageTemplate, params object[] propertyValues); + void LogWarning(Exception exception, string messageTemplate, params object[] propertyValues); /// /// Logs an information message. diff --git a/src/Umbraco.Core/Logging/LoggerExtensions.cs b/src/Umbraco.Core/Logging/LoggerExtensions.cs index ef006edd21..515fed9db1 100644 --- a/src/Umbraco.Core/Logging/LoggerExtensions.cs +++ b/src/Umbraco.Core/Logging/LoggerExtensions.cs @@ -24,7 +24,7 @@ namespace Umbraco.Core.Logging /// A message. /// An exception. public static void LogError(this ILogger logger, Exception exception, string message) - => logger.LogError(typeof(T), exception, message); + => logger.LogError(exception, message); /// /// Logs an error message with an exception. @@ -35,16 +35,7 @@ namespace Umbraco.Core.Logging /// A message template. /// Property values. public static void LogError(this ILogger logger, Exception exception, string messageTemplate, params object[] propertyValues) - => logger.LogError(typeof(T), exception, messageTemplate, propertyValues); - - /// - /// Logs an error exception. - /// - /// The reporting type. - /// The logger. - /// An exception. - public static void LogError(this ILogger logger, Exception exception) - => logger.LogError(typeof(T), exception); + => logger.LogError(exception, messageTemplate, propertyValues); /// /// Logs an error message. @@ -53,7 +44,7 @@ namespace Umbraco.Core.Logging /// The logger. /// A message. public static void LogError(this ILogger logger, string message) - => logger.LogError(typeof(T), message); + => logger.LogError(message); /// /// Logs an error message. @@ -63,7 +54,7 @@ namespace Umbraco.Core.Logging /// A message template. /// Property values. public static void LogError(this ILogger logger, string messageTemplate, params object[] propertyValues) - => logger.LogError(typeof(T), messageTemplate, propertyValues); + => logger.LogError(messageTemplate, propertyValues); /// /// Logs a warning message. @@ -72,7 +63,7 @@ namespace Umbraco.Core.Logging /// The logger. /// A message. public static void LogWarning(this ILogger logger, string message) - => logger.LogWarning(typeof(T), message); + => logger.LogWarning(message); /// /// Logs a warning message. @@ -82,7 +73,7 @@ namespace Umbraco.Core.Logging /// A message template. /// Property values. public static void LogWarning(this ILogger logger, string messageTemplate, params object[] propertyValues) - => logger.LogWarning(typeof(T), messageTemplate, propertyValues); + => logger.LogWarning(messageTemplate, propertyValues); /// /// Logs a warning message with an exception. @@ -92,7 +83,7 @@ namespace Umbraco.Core.Logging /// An exception. /// A message. public static void LogWarning(this ILogger logger, Exception exception, string message) - => logger.LogWarning(typeof(T), exception, message); + => logger.LogWarning(exception, message); /// /// Logs a warning message with an exception. @@ -103,7 +94,7 @@ namespace Umbraco.Core.Logging /// A message template. /// Property values. public static void LogWarning(this ILogger logger, Exception exception, string messageTemplate, params object[] propertyValues) - => logger.LogWarning(typeof(T), exception, messageTemplate, propertyValues); + => logger.LogWarning(exception, messageTemplate, propertyValues); /// /// Logs an information message. @@ -170,7 +161,7 @@ namespace Umbraco.Core.Logging /// An exception. /// A message. public static void Fatal(this ILogger logger, Exception exception, string message) - => logger.LogCritical(exception, message, new object[] { }); + => logger.LogCritical(exception, message); /// /// Logs a fatal message. diff --git a/src/Umbraco.Core/Logging/NullLogger.cs b/src/Umbraco.Core/Logging/NullLogger.cs index 302fe46592..902da4c9c9 100644 --- a/src/Umbraco.Core/Logging/NullLogger.cs +++ b/src/Umbraco.Core/Logging/NullLogger.cs @@ -21,47 +21,27 @@ namespace Umbraco.Core.Logging } - public void LogError(Type reporting, Exception exception, string message) - { - - } - public void LogError(Type reporting, Exception exception) { } - public void LogError(Type reporting, string message) + public void LogError(string message, params object[] propertyValues) { } - public void LogError(Type reporting, Exception exception, string messageTemplate, params object[] propertyValues) + public void LogError(Exception exception, string messageTemplate, params object[] propertyValues) { } - public void LogError(Type reporting, string messageTemplate, params object[] propertyValues) + public void LogWarning(string messageTemplate, params object[] propertyValues) { } - public void LogWarning(Type reporting, string message) - { - - } - - public void LogWarning(Type reporting, string messageTemplate, params object[] propertyValues) - { - - } - - public void LogWarning(Type reporting, Exception exception, string message) - { - - } - - public void LogWarning(Type reporting, Exception exception, string messageTemplate, params object[] propertyValues) + public void LogWarning(Exception exception, string messageTemplate, params object[] propertyValues) { } diff --git a/src/Umbraco.Core/Logging/ProfilingLogger.cs b/src/Umbraco.Core/Logging/ProfilingLogger.cs index 9670874d65..331d9f2b36 100644 --- a/src/Umbraco.Core/Logging/ProfilingLogger.cs +++ b/src/Umbraco.Core/Logging/ProfilingLogger.cs @@ -67,41 +67,23 @@ namespace Umbraco.Core.Logging public bool IsEnabled(Type reporting, LogLevel level) => Logger.IsEnabled(reporting, level); - public void Fatal(Type reporting, Exception exception) - => Logger.Fatal(reporting, exception); - public void LogCritical(Exception exception, string messageTemplate, params object[] propertyValues) => Logger.LogCritical(exception, messageTemplate, propertyValues); public void LogCritical(string messageTemplate, params object[] propertyValues) => Logger.LogCritical(messageTemplate, propertyValues); - public void LogError(Type reporting, Exception exception, string message) - => Logger.LogError(reporting, exception, message); + public void LogError(Exception exception, string messageTemplate, params object[] propertyValues) + => Logger.LogError(exception, messageTemplate, propertyValues); - public void LogError(Type reporting, Exception exception) - => Logger.LogError(reporting, exception); + public void LogError(string messageTemplate, params object[] propertyValues) + => Logger.LogError(messageTemplate, propertyValues); - public void LogError(Type reporting, string message) - => Logger.LogError(reporting, message); + public void LogWarning(string messageTemplate, params object[] propertyValues) + => Logger.LogWarning(messageTemplate, propertyValues); - public void LogError(Type reporting, Exception exception, string messageTemplate, params object[] propertyValues) - => Logger.LogError(reporting, exception, messageTemplate, propertyValues); - - public void LogError(Type reporting, string messageTemplate, params object[] propertyValues) - => Logger.LogError(reporting, messageTemplate, propertyValues); - - public void LogWarning(Type reporting, string message) - => Logger.LogWarning(reporting, message); - - public void LogWarning(Type reporting, string messageTemplate, params object[] propertyValues) - => Logger.LogWarning(reporting, messageTemplate, propertyValues); - - public void LogWarning(Type reporting, Exception exception, string message) - => Logger.LogWarning(reporting, exception, message); - - public void LogWarning(Type reporting, Exception exception, string messageTemplate, params object[] propertyValues) - => Logger.LogWarning(reporting, exception, messageTemplate, propertyValues); + public void LogWarning(Exception exception, string messageTemplate, params object[] propertyValues) + => Logger.LogWarning(exception, messageTemplate, propertyValues); public void Info(Type reporting, string message) => Logger.Info(reporting, message); diff --git a/src/Umbraco.Examine.Lucene/LuceneIndexDiagnostics.cs b/src/Umbraco.Examine.Lucene/LuceneIndexDiagnostics.cs index e4d6460eb4..341eb67772 100644 --- a/src/Umbraco.Examine.Lucene/LuceneIndexDiagnostics.cs +++ b/src/Umbraco.Examine.Lucene/LuceneIndexDiagnostics.cs @@ -34,7 +34,7 @@ namespace Umbraco.Examine } catch (AlreadyClosedException) { - Logger.LogWarning(typeof(UmbracoContentIndex), "Cannot get GetIndexDocumentCount, the writer is already closed"); + Logger.LogWarning("Cannot get GetIndexDocumentCount, the writer is already closed"); return 0; } } @@ -50,7 +50,7 @@ namespace Umbraco.Examine } catch (AlreadyClosedException) { - Logger.LogWarning(typeof(UmbracoContentIndex), "Cannot get GetIndexFieldCount, the writer is already closed"); + Logger.LogWarning("Cannot get GetIndexFieldCount, the writer is already closed"); return 0; } } diff --git a/src/Umbraco.Examine.Lucene/UmbracoExamineIndex.cs b/src/Umbraco.Examine.Lucene/UmbracoExamineIndex.cs index b904ce1f36..10a1d075b0 100644 --- a/src/Umbraco.Examine.Lucene/UmbracoExamineIndex.cs +++ b/src/Umbraco.Examine.Lucene/UmbracoExamineIndex.cs @@ -118,7 +118,7 @@ namespace Umbraco.Examine /// protected override void OnIndexingError(IndexingErrorEventArgs ex) { - ProfilingLogger.LogError(GetType(), ex.InnerException, ex.Message); + ProfilingLogger.LogError(ex.InnerException, ex.Message); base.OnIndexingError(ex); } diff --git a/src/Umbraco.Infrastructure/Compose/NotificationsComponent.cs b/src/Umbraco.Infrastructure/Compose/NotificationsComponent.cs index 6dd265483d..49c34331c4 100644 --- a/src/Umbraco.Infrastructure/Compose/NotificationsComponent.cs +++ b/src/Umbraco.Infrastructure/Compose/NotificationsComponent.cs @@ -222,7 +222,7 @@ namespace Umbraco.Web.Compose user = _userService.GetUserById(Constants.Security.SuperUserId); if (user == null) { - _logger.LogWarning(typeof(Notifier), "Notifications can not be sent, no admin user with id {SuperUserId} could be resolved", Constants.Security.SuperUserId); + _logger.LogWarning("Notifications can not be sent, no admin user with id {SuperUserId} could be resolved", Constants.Security.SuperUserId); return; } } @@ -235,7 +235,7 @@ namespace Umbraco.Web.Compose if (sender == null) throw new ArgumentNullException(nameof(sender)); if (siteUri == null) { - _logger.LogWarning(typeof(Notifier), "Notifications can not be sent, no site url is set (might be during boot process?)"); + _logger.LogWarning("Notifications can not be sent, no site url is set (might be during boot process?)"); return; } diff --git a/src/Umbraco.Infrastructure/Logging/Serilog/SerilogLogger.cs b/src/Umbraco.Infrastructure/Logging/Serilog/SerilogLogger.cs index dac7e86f51..00a5ff918f 100644 --- a/src/Umbraco.Infrastructure/Logging/Serilog/SerilogLogger.cs +++ b/src/Umbraco.Infrastructure/Logging/Serilog/SerilogLogger.cs @@ -80,14 +80,6 @@ namespace Umbraco.Core.Logging.Serilog public bool IsEnabled(Type reporting, LogLevel level) => LoggerFor(reporting).IsEnabled(MapLevel(level)); - /// - public void Fatal(Type reporting, Exception exception) - { - var logger = LoggerFor(reporting); - var message = "Exception."; - logger.Fatal(exception, message); - } - /// public void LogCritical(string messageTemplate, params object[] propertyValues) { @@ -101,13 +93,6 @@ namespace Umbraco.Core.Logging.Serilog logger.Fatal(exception, messageTemplate, propertyValues); } - /// - public void LogError(Type reporting, Exception exception, string message) - { - var logger = LoggerFor(reporting); - logger.Error(exception, message); - } - /// public void LogError(Type reporting, Exception exception) { @@ -117,46 +102,28 @@ namespace Umbraco.Core.Logging.Serilog } /// - public void LogError(Type reporting, string message) + public void LogError(string messageTemplate, params object[] propertyValues) { - LoggerFor(reporting).Error(message); + LoggerFor(typeof(T)).Error(messageTemplate, propertyValues); } /// - public void LogError(Type reporting, string messageTemplate, params object[] propertyValues) + public void LogError(Exception exception, string messageTemplate, params object[] propertyValues) { - LoggerFor(reporting).Error(messageTemplate, propertyValues); - } - - /// - public void LogError(Type reporting, Exception exception, string messageTemplate, params object[] propertyValues) - { - var logger = LoggerFor(reporting); + var logger = LoggerFor(typeof(T)); logger.Error(exception, messageTemplate, propertyValues); } /// - public void LogWarning(Type reporting, string message) + public void LogWarning(string message, params object[] propertyValues) { - LoggerFor(reporting).Warning(message); + LoggerFor(typeof(T)).Warning(message, propertyValues); } /// - public void LogWarning(Type reporting, string message, params object[] propertyValues) + public void LogWarning(Exception exception, string messageTemplate, params object[] propertyValues) { - LoggerFor(reporting).Warning(message, propertyValues); - } - - /// - public void LogWarning(Type reporting, Exception exception, string message) - { - LoggerFor(reporting).Warning(exception, message); - } - - /// - public void LogWarning(Type reporting, Exception exception, string messageTemplate, params object[] propertyValues) - { - LoggerFor(reporting).Warning(exception, messageTemplate, propertyValues); + LoggerFor(typeof(T)).Warning(exception, messageTemplate, propertyValues); } /// diff --git a/src/Umbraco.Infrastructure/Media/UploadAutoFillProperties.cs b/src/Umbraco.Infrastructure/Media/UploadAutoFillProperties.cs index c675fa4079..e947afe6e2 100644 --- a/src/Umbraco.Infrastructure/Media/UploadAutoFillProperties.cs +++ b/src/Umbraco.Infrastructure/Media/UploadAutoFillProperties.cs @@ -74,7 +74,7 @@ namespace Umbraco.Web.Media } catch (Exception ex) { - _logger.LogError(typeof(UploadAutoFillProperties), ex, "Could not populate upload auto-fill properties for file '{File}'.", filepath); + _logger.LogError(ex, "Could not populate upload auto-fill properties for file '{File}'.", filepath); ResetProperties(content, autoFillConfig, culture, segment); } } diff --git a/src/Umbraco.Infrastructure/Migrations/Upgrade/V_8_0_0/PropertyEditorsMigrationBase.cs b/src/Umbraco.Infrastructure/Migrations/Upgrade/V_8_0_0/PropertyEditorsMigrationBase.cs index 554402fe38..61e69fe234 100644 --- a/src/Umbraco.Infrastructure/Migrations/Upgrade/V_8_0_0/PropertyEditorsMigrationBase.cs +++ b/src/Umbraco.Infrastructure/Migrations/Upgrade/V_8_0_0/PropertyEditorsMigrationBase.cs @@ -78,7 +78,7 @@ namespace Umbraco.Core.Migrations.Upgrade.V_8_0_0 continue; } - Logger.LogWarning(GetType(), "Could not find PropertyData {PropertyDataId} value '{PropertyValue}' in the datatype configuration: {Values}.", + Logger.LogWarning("Could not find PropertyData {PropertyDataId} value '{PropertyValue}' in the datatype configuration: {Values}.", propData.Id, id, string.Join(", ", config.Items.Select(x => x.Id + ":" + x.Value))); canConvert = false; } diff --git a/src/Umbraco.Infrastructure/Models/Mapping/PropertyTypeGroupMapper.cs b/src/Umbraco.Infrastructure/Models/Mapping/PropertyTypeGroupMapper.cs index 99708d3fad..e67fba2206 100644 --- a/src/Umbraco.Infrastructure/Models/Mapping/PropertyTypeGroupMapper.cs +++ b/src/Umbraco.Infrastructure/Models/Mapping/PropertyTypeGroupMapper.cs @@ -210,8 +210,7 @@ namespace Umbraco.Web.Models.Mapping //fixme: Don't explode if we can't find this, log an error and change this to a label if (propertyEditor == null) { - _logger.LogError(GetType(), - "No property editor could be resolved with the alias: {PropertyEditorAlias}, defaulting to label", p.PropertyEditorAlias); + _logger.LogError("No property editor could be resolved with the alias: {PropertyEditorAlias}, defaulting to label", p.PropertyEditorAlias); propertyEditorAlias = Constants.PropertyEditors.Aliases.Label; propertyEditor = _propertyEditors[propertyEditorAlias]; } diff --git a/src/Umbraco.Infrastructure/Models/PathValidationExtensions.cs b/src/Umbraco.Infrastructure/Models/PathValidationExtensions.cs index ed027026bc..722bd8fd85 100644 --- a/src/Umbraco.Infrastructure/Models/PathValidationExtensions.cs +++ b/src/Umbraco.Infrastructure/Models/PathValidationExtensions.cs @@ -88,7 +88,7 @@ namespace Umbraco.Core.Models if (entity.ValidatePath() == false) { - logger.LogWarning(typeof(PathValidationExtensions), "The content item {EntityId} has an invalid path: {EntityPath} with parentID: {EntityParentId}", entity.Id, entity.Path, entity.ParentId); + logger.LogWarning("The content item {EntityId} has an invalid path: {EntityPath} with parentID: {EntityParentId}", entity.Id, entity.Path, entity.ParentId); if (entity.ParentId == -1) { entity.Path = string.Concat("-1,", entity.Id); diff --git a/src/Umbraco.Infrastructure/Models/PropertyTagsExtensions.cs b/src/Umbraco.Infrastructure/Models/PropertyTagsExtensions.cs index f2e4bd1074..ff3ad57585 100644 --- a/src/Umbraco.Infrastructure/Models/PropertyTagsExtensions.cs +++ b/src/Umbraco.Infrastructure/Models/PropertyTagsExtensions.cs @@ -218,7 +218,7 @@ namespace Umbraco.Core.Models } catch (Exception ex) { - Current.Logger.LogWarning(typeof(PropertyTagsExtensions), ex, "Could not automatically convert stored json value to an enumerable string '{Json}'", value.ToString()); + Current.Logger.LogWarning(ex, "Could not automatically convert stored json value to an enumerable string '{Json}'", value.ToString()); } break; diff --git a/src/Umbraco.Infrastructure/Persistence/Factories/DataTypeFactory.cs b/src/Umbraco.Infrastructure/Persistence/Factories/DataTypeFactory.cs index 71186bfea5..e447233493 100644 --- a/src/Umbraco.Infrastructure/Persistence/Factories/DataTypeFactory.cs +++ b/src/Umbraco.Infrastructure/Persistence/Factories/DataTypeFactory.cs @@ -13,8 +13,8 @@ namespace Umbraco.Core.Persistence.Factories // Check we have an editor for the data type. if (!editors.TryGet(dto.EditorAlias, out var editor)) { - logger.LogWarning(typeof(DataType), "Could not find an editor with alias {EditorAlias}, treating as Label. " + - "The site may fail to boot and/or load data types and run.", dto.EditorAlias); + logger.LogWarning("Could not find an editor with alias {EditorAlias}, treating as Label. " + + "The site may fail to boot and/or load data types and run.", dto.EditorAlias); // Create as special type, which downstream can be handled by converting to a LabelPropertyEditor to make clear // the situation to the user. diff --git a/src/Umbraco.Infrastructure/PropertyEditors/RichTextEditorPastedImages.cs b/src/Umbraco.Infrastructure/PropertyEditors/RichTextEditorPastedImages.cs index 81b3ba7a63..d7d795c175 100644 --- a/src/Umbraco.Infrastructure/PropertyEditors/RichTextEditorPastedImages.cs +++ b/src/Umbraco.Infrastructure/PropertyEditors/RichTextEditorPastedImages.cs @@ -144,7 +144,7 @@ namespace Umbraco.Web.PropertyEditors } catch (Exception ex) { - _logger.LogError(typeof(HtmlImageSourceParser), ex, "Could not delete temp file or folder {FileName}", absoluteTempImagePath); + _logger.LogError(ex, "Could not delete temp file or folder {FileName}", absoluteTempImagePath); } } } diff --git a/src/Umbraco.Infrastructure/Runtime/MainDomSemaphoreLock.cs b/src/Umbraco.Infrastructure/Runtime/MainDomSemaphoreLock.cs index 08e038a501..50c4b91671 100644 --- a/src/Umbraco.Infrastructure/Runtime/MainDomSemaphoreLock.cs +++ b/src/Umbraco.Infrastructure/Runtime/MainDomSemaphoreLock.cs @@ -52,7 +52,7 @@ namespace Umbraco.Core.Runtime } catch (TimeoutException ex) { - _logger.LogError(ex); + _logger.LogError(ex.Message); return Task.FromResult(false); } finally diff --git a/src/Umbraco.Web.BackOffice/Controllers/DictionaryController.cs b/src/Umbraco.Web.BackOffice/Controllers/DictionaryController.cs index 882abd7440..f763deae0a 100644 --- a/src/Umbraco.Web.BackOffice/Controllers/DictionaryController.cs +++ b/src/Umbraco.Web.BackOffice/Controllers/DictionaryController.cs @@ -124,7 +124,7 @@ namespace Umbraco.Web.BackOffice.Controllers } catch (Exception ex) { - _logger.LogError(GetType(), ex, "Error creating dictionary with {Name} under {ParentId}", key, parentId); + _logger.LogError(ex, "Error creating dictionary with {Name} under {ParentId}", key, parentId); throw HttpResponseException.CreateNotificationValidationErrorResponse("Error creating dictionary item"); } } @@ -257,7 +257,7 @@ namespace Umbraco.Web.BackOffice.Controllers } catch (Exception ex) { - _logger.LogError(GetType(), ex, "Error saving dictionary with {Name} under {ParentId}", dictionary.Name, dictionary.ParentId); + _logger.LogError(ex, "Error saving dictionary with {Name} under {ParentId}", dictionary.Name, dictionary.ParentId); throw HttpResponseException.CreateNotificationValidationErrorResponse("Something went wrong saving dictionary"); } } diff --git a/src/Umbraco.Web.BackOffice/Controllers/RelationTypeController.cs b/src/Umbraco.Web.BackOffice/Controllers/RelationTypeController.cs index b67a2f8b48..8b57703a9f 100644 --- a/src/Umbraco.Web.BackOffice/Controllers/RelationTypeController.cs +++ b/src/Umbraco.Web.BackOffice/Controllers/RelationTypeController.cs @@ -154,7 +154,7 @@ namespace Umbraco.Web.BackOffice.Controllers } catch (Exception ex) { - _logger.LogError(GetType(), ex, "Error creating relation type with {Name}", relationType.Name); + _logger.LogError(ex, "Error creating relation type with {Name}", relationType.Name); throw HttpResponseException.CreateNotificationValidationErrorResponse("Error creating relation type."); } } @@ -185,7 +185,7 @@ namespace Umbraco.Web.BackOffice.Controllers } catch (Exception ex) { - _logger.LogError(GetType(), ex, "Error saving relation type with {Id}", relationType.Id); + _logger.LogError(ex, "Error saving relation type with {Id}", relationType.Id); throw HttpResponseException.CreateNotificationValidationErrorResponse("Something went wrong when saving the relation type"); } } diff --git a/src/Umbraco.Web.Common/Extensions/ImageCropperTemplateExtensions.cs b/src/Umbraco.Web.Common/Extensions/ImageCropperTemplateExtensions.cs index 2cd90e83db..7808c83295 100644 --- a/src/Umbraco.Web.Common/Extensions/ImageCropperTemplateExtensions.cs +++ b/src/Umbraco.Web.Common/Extensions/ImageCropperTemplateExtensions.cs @@ -27,7 +27,7 @@ namespace Umbraco.Extensions } catch (Exception ex) { - Current.Logger.LogError(typeof(ImageCropperTemplateExtensions), ex, "Could not parse the json string: {Json}", json); + Current.Logger.LogError(ex, "Could not parse the json string: {Json}", json); } } diff --git a/src/Umbraco.Web.Common/Install/InstallController.cs b/src/Umbraco.Web.Common/Install/InstallController.cs index e9c8de2a83..7840a1097b 100644 --- a/src/Umbraco.Web.Common/Install/InstallController.cs +++ b/src/Umbraco.Web.Common/Install/InstallController.cs @@ -118,7 +118,7 @@ namespace Umbraco.Web.Common.Install if (_reported && _reportedLevel == level) return; _reported = true; _reportedLevel = level; - logger.LogWarning(typeof(UmbracoInstallApplicationBuilderExtensions), message); + logger.LogWarning(message); } } } diff --git a/src/Umbraco.Web.Common/Middleware/UmbracoRequestMiddleware.cs b/src/Umbraco.Web.Common/Middleware/UmbracoRequestMiddleware.cs index 67334e941d..e38c91b1f5 100644 --- a/src/Umbraco.Web.Common/Middleware/UmbracoRequestMiddleware.cs +++ b/src/Umbraco.Web.Common/Middleware/UmbracoRequestMiddleware.cs @@ -65,7 +65,7 @@ namespace Umbraco.Web.Common.Middleware catch (Exception ex) { // try catch so we don't kill everything in all requests - _logger.LogError(ex); + _logger.LogError(ex.Message); } finally { diff --git a/src/Umbraco.Web/ImageProcessorLogger.cs b/src/Umbraco.Web/ImageProcessorLogger.cs index 8f6be923ef..78c0cd231e 100644 --- a/src/Umbraco.Web/ImageProcessorLogger.cs +++ b/src/Umbraco.Web/ImageProcessorLogger.cs @@ -28,7 +28,7 @@ namespace Umbraco.Web { // Using LogHelper since the ImageProcessor logger expects a parameterless constructor. var message = $"{callerName} {lineNumber} : {text}"; - Current.Logger.LogError(new ImageProcessingException(message)); + Current.Logger.LogError(new ImageProcessingException(message).Message); } /// @@ -42,7 +42,7 @@ namespace Umbraco.Web { // Using LogHelper since the ImageProcessor logger expects a parameterless constructor. var message = $"{callerName} {lineNumber} : {text}"; - Current.Logger.LogError(type, new ImageProcessingException(message)); + Current.Logger.LogError(new ImageProcessingException(message).Message); } } } diff --git a/src/Umbraco.Web/Logging/OwinLogger.cs b/src/Umbraco.Web/Logging/OwinLogger.cs index dcd1a0f531..dcbbb3506e 100644 --- a/src/Umbraco.Web/Logging/OwinLogger.cs +++ b/src/Umbraco.Web/Logging/OwinLogger.cs @@ -30,10 +30,10 @@ namespace Umbraco.Web.Logging _logger.LogCritical(exception, "[{EventType}] Event Id: {EventId}, State: {State}", eventType, eventId, state); return true; case TraceEventType.Error: - _logger.LogError(_type.Value, exception, "[{EventType}] Event Id: {EventId}, State: {State}", eventType, eventId, state); + _logger.LogError(exception, "[{EventType}] Event Id: {EventId}, State: {State}", eventType, eventId, state); return true; case TraceEventType.Warning: - _logger.LogWarning(_type.Value, "[{EventType}] Event Id: {EventId}, State: {State}", eventType, eventId, state); + _logger.LogWarning("[{EventType}] Event Id: {EventId}, State: {State}", eventType, eventId, state); return true; case TraceEventType.Information: _logger.Info(_type.Value, "[{EventType}] Event Id: {EventId}, State: {State}", eventType, eventId, state); diff --git a/src/Umbraco.Web/Security/AuthenticationOptionsExtensions.cs b/src/Umbraco.Web/Security/AuthenticationOptionsExtensions.cs index 48a13f265d..660007917c 100644 --- a/src/Umbraco.Web/Security/AuthenticationOptionsExtensions.cs +++ b/src/Umbraco.Web/Security/AuthenticationOptionsExtensions.cs @@ -109,7 +109,7 @@ namespace Umbraco.Web.Security } catch (System.Exception ex) { - Current.Logger.LogError(typeof (AuthenticationOptionsExtensions), ex, "Could not read AuthenticationOptions properties"); + Current.Logger.LogError(ex, "Could not read AuthenticationOptions properties"); } } else diff --git a/src/Umbraco.Web/UmbracoHelper.cs b/src/Umbraco.Web/UmbracoHelper.cs index 78a296ad39..93b1acfbe4 100644 --- a/src/Umbraco.Web/UmbracoHelper.cs +++ b/src/Umbraco.Web/UmbracoHelper.cs @@ -455,7 +455,7 @@ namespace Umbraco.Web } catch (Exception ex) when (ex is FormatException || ex is ArgumentException) { - Current.Logger.LogWarning(typeof(UmbracoHelper), "A value was detected in the ufprt parameter but Umbraco could not decrypt the string"); + Current.Logger.LogWarning("A value was detected in the ufprt parameter but Umbraco could not decrypt the string"); parts = null; return false; } diff --git a/src/Umbraco.Web/WebApi/Filters/AngularAntiForgeryHelper.cs b/src/Umbraco.Web/WebApi/Filters/AngularAntiForgeryHelper.cs index 98e518e7d0..717df89a7b 100644 --- a/src/Umbraco.Web/WebApi/Filters/AngularAntiForgeryHelper.cs +++ b/src/Umbraco.Web/WebApi/Filters/AngularAntiForgeryHelper.cs @@ -44,7 +44,7 @@ namespace Umbraco.Web.WebApi.Filters } catch (Exception ex) { - Current.Logger.LogError(typeof(AngularAntiForgeryHelper), ex, "Could not validate XSRF token"); + Current.Logger.LogError(ex, "Could not validate XSRF token"); return false; } return true; diff --git a/src/Umbraco.Web/WebApi/UnhandledExceptionLogger.cs b/src/Umbraco.Web/WebApi/UnhandledExceptionLogger.cs index 970098e957..3803cdbab6 100644 --- a/src/Umbraco.Web/WebApi/UnhandledExceptionLogger.cs +++ b/src/Umbraco.Web/WebApi/UnhandledExceptionLogger.cs @@ -28,7 +28,7 @@ namespace Umbraco.Web.WebApi var requestUrl = context.ExceptionContext?.ControllerContext?.Request?.RequestUri?.AbsoluteUri; var controllerType = context.ExceptionContext?.ActionContext?.ControllerContext?.Controller?.GetType(); - _logger.LogError(controllerType, context.Exception, "Unhandled controller exception occurred for request '{RequestUrl}'", requestUrl); + _logger.LogError(context.Exception, "Unhandled controller exception occurred for request '{RequestUrl}'", requestUrl); } }