From e3e9b3624c7d8fc078dc877dc19f586ac7c94892 Mon Sep 17 00:00:00 2001
From: Julian Verdurmen <304NotModified@users.noreply.github.com>
Date: Thu, 12 Oct 2017 18:43:28 +0200
Subject: [PATCH] add ConfigureNLog on ILoggingBuilder

---
 NLog.Web.AspNetCore/AspNetExtensions.cs | 29 +++++++++++++++++++++++--
 NLog.Web.AspNetCore/NLogBuilder.cs      | 12 +++++-----
 2 files changed, 33 insertions(+), 8 deletions(-)

diff --git a/NLog.Web.AspNetCore/AspNetExtensions.cs b/NLog.Web.AspNetCore/AspNetExtensions.cs
index efe72729..432d7b2c 100644
--- a/NLog.Web.AspNetCore/AspNetExtensions.cs
+++ b/NLog.Web.AspNetCore/AspNetExtensions.cs
@@ -62,6 +62,31 @@ private static LoggingConfiguration ConfigureNLog(string fileName)
 
 #if NETSTANDARD2_0
 
+        /// <summary>
+        /// Apply NLog configuration from XML config.
+        /// 
+        /// This call is not needed when <see cref="NLogBuilder.ConfigureNLog(string)"/> is used.
+        /// </summary>
+        /// <param name="builder">The logging builder</param>
+        /// <param name="configFileName">Path to NLog configuration file, e.g. nlog.config. </param>>
+        /// <returns>LogFactory to get loggers, add events etc</returns>
+        public static LogFactory ConfigureNLog(this ILoggingBuilder builder, string configFileName)
+        {
+            return NLogBuilder.ConfigureNLog(configFileName);
+        }
+
+        /// <summary>
+        /// Configure NLog from API
+        /// 
+        /// This call is not needed when <see cref="NLogBuilder.ConfigureNLog(LoggingConfiguration)"/> is used.
+        /// </summary>
+        /// <param name="builder">The logging builder</param>
+        /// <param name="configuration">Config for NLog</param>
+        /// <returns>LogFactory to get loggers, add events etc</returns>
+        public static LogFactory ConfigureNLog(this ILoggingBuilder builder, LoggingConfiguration configuration)
+        {
+            return NLogBuilder.ConfigureNLog(configuration);
+        }
 
         /// <summary>
         /// Use NLog for Dependency Injected loggers. 
@@ -89,7 +114,7 @@ public static IWebHostBuilder UseNLog(this IWebHostBuilder builder, NLogAspNetCo
                     ServiceLocator.ServiceProvider = serviceProvider;
 
                     NLogBuilder.RegisterNLogWebAspNetCore();
-                    
+
                     LogManager.Configuration?.Reload();
                     return new NLogLoggerFactory(options);
 
@@ -100,7 +125,7 @@ public static IWebHostBuilder UseNLog(this IWebHostBuilder builder, NLogAspNetCo
                     services.TryAddSingleton<IHttpContextAccessor, HttpContextAccessor>();
                 }
 
-                
+
 
 
 
diff --git a/NLog.Web.AspNetCore/NLogBuilder.cs b/NLog.Web.AspNetCore/NLogBuilder.cs
index 414d7069..92117d54 100644
--- a/NLog.Web.AspNetCore/NLogBuilder.cs
+++ b/NLog.Web.AspNetCore/NLogBuilder.cs
@@ -34,20 +34,20 @@ internal static void RegisterNLogWebAspNetCore()
         }
 
         /// <summary>
-        /// Configure NLog
+        /// Configure NLog from XML config.
         /// </summary>
-        /// <param name="fileName">Path to NLog configuration file, e.g. nlog.config. </param>>
+        /// <param name="configFileName">Path to NLog configuration file, e.g. nlog.config. </param>>
         /// <returns>LogFactory to get loggers, add events etc</returns>
-        public static LogFactory ConfigureNLog(string fileName)
+        public static LogFactory ConfigureNLog(string configFileName)
         {
             RegisterNLogWebAspNetCore();
-            return ConfigureNLog(new XmlLoggingConfiguration(fileName));
+            return ConfigureNLog(new XmlLoggingConfiguration(configFileName));
         }
 
         /// <summary>
-        /// Configure NLog
+        /// Configure NLog from API
         /// </summary>
-        /// <param name="configuration"></param>
+        /// <param name="configuration">Config for NLog</param>
         /// <returns>LogFactory to get loggers, add events etc</returns>
         public static LogFactory ConfigureNLog(LoggingConfiguration configuration)
         {