diff --git a/CHANGELOG.md b/CHANGELOG.md index 6d9eef1e02..ef822cedc8 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -19,6 +19,7 @@ Deprecations: Enhancements: - Recognize read-only parameters by the `In` modreq (@zvirja, #406) + - NLogFactory and ExtendedNLogFactory default constructor will no longer load NLog.config, 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) diff --git a/src/Castle.Services.Logging.NLogIntegration/NLogFactory.cs b/src/Castle.Services.Logging.NLogIntegration/NLogFactory.cs index 90db42ef78..6b9547d70a 100644 --- a/src/Castle.Services.Logging.NLogIntegration/NLogFactory.cs +++ b/src/Castle.Services.Logging.NLogIntegration/NLogFactory.cs @@ -32,14 +32,14 @@ public class NLogFactory : AbstractLoggerFactory /// Initializes a new instance of the class. /// public NLogFactory() - : this(defaultConfigFileName) + : this(true) { } /// /// 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 NLogFactory(bool configuredExternally) { if (configuredExternally)