From f18057df2689ac8f4b398bd491d27e4335db45fa Mon Sep 17 00:00:00 2001 From: Rolf Kristensen Date: Mon, 12 Nov 2018 23:54:52 +0100 Subject: [PATCH] ExtendedNLogFactory - Removed explicit loading of nlog.config by default, since NLog will automatically load its configuration --- CHANGELOG.md | 2 +- .../ExtendedNLogFactory.cs | 9 +++------ 2 files changed, 4 insertions(+), 7 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 8752b0b6fd..3249f2ca61 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -19,7 +19,7 @@ Deprecations: Enhancements: - Recognize read-only parameters by the `In` modreq (@zvirja, #406) - - NLogFactory will no longer load NLog.config by default, since NLog does it automatically (@snakefoot, #419) + - NLogFactory and ExtendedNLogFactory will no longer load NLog.config by default, since NLog does it automatically (@snakefoot, #419) ## 4.3.1 (2018-06-21) diff --git a/src/Castle.Services.Logging.NLogIntegration/ExtendedNLogFactory.cs b/src/Castle.Services.Logging.NLogIntegration/ExtendedNLogFactory.cs index ccf202ebc0..c4a681f76c 100644 --- a/src/Castle.Services.Logging.NLogIntegration/ExtendedNLogFactory.cs +++ b/src/Castle.Services.Logging.NLogIntegration/ExtendedNLogFactory.cs @@ -28,17 +28,14 @@ public class ExtendedNLogFactory : AbstractExtendedLoggerFactory { /// /// Initializes a new instance of the class. - /// Configures NLog with a config file name 'nlog.config' - /// /// public ExtendedNLogFactory() - : this(NLogFactory.defaultConfigFileName) + : this(true) { } /// - /// Initializes a new instance of the class with the configfile specified by + /// Initializes a new instance of the class. /// /// The config file. public ExtendedNLogFactory(string configFile) @@ -50,7 +47,7 @@ public ExtendedNLogFactory(string configFile) /// /// Initializes a new instance of the class. /// - /// If true . Skips the initialization of log4net assuming it will happen externally. Useful if you're using another framework that wants to take over configuration of NLog. + /// If true. Skips the initialization of NLog assuming it will happen externally (or automatically by NLog). public ExtendedNLogFactory(bool configuredExternally) { if (configuredExternally)