Merge pull request #159 from NLog/fix-Recursive-resource-lookup-bug

Fix Recursive resource lookup bug

Author: 304NotModified

src/NLog.Extensions.Logging/ConfigureExtensions.cs

@@ -2,6 +2,7 @@
 using System.IO;
 using System.Reflection;
 using Microsoft.Extensions.Logging;
+using NLog.Common;
 using NLog.Config;
 
 namespace NLog.Extensions.Logging
@@ -71,22 +72,30 @@ public static ILoggingBuilder AddNLog(this ILoggingBuilder factory, NLogProvider
 
         private static void ConfigureHiddenAssemblies()
         {
-            //ignore this
-            LogManager.AddHiddenAssembly(Assembly.Load(new AssemblyName("Microsoft.Extensions.Logging")));
-            LogManager.AddHiddenAssembly(Assembly.Load(new AssemblyName("Microsoft.Extensions.Logging.Abstractions")));
-
             try
             {
-                //try the Filter ext
-                var filterAssembly = Assembly.Load(new AssemblyName("Microsoft.Extensions.Logging.Filter"));
-                LogManager.AddHiddenAssembly(filterAssembly);
+                //ignore these assemblies for ${callsite}
+                LogManager.AddHiddenAssembly(typeof(Microsoft.Extensions.Logging.LoggerFactoryExtensions).GetTypeInfo().Assembly); //Microsoft.Extensions.Logging
+                LogManager.AddHiddenAssembly(typeof(Microsoft.Extensions.Logging.ILogger).GetTypeInfo().Assembly); // Microsoft.Extensions.Logging.Abstractions
+                LogManager.AddHiddenAssembly(typeof(NLog.Extensions.Logging.ConfigureExtensions).GetTypeInfo().Assembly); //NLog.Extensions.Logging
+
+                try
+                {
+                    //try the Filter ext
+                    var filterAssembly = Assembly.Load(new AssemblyName("Microsoft.Extensions.Logging.Filter"));
+                    LogManager.AddHiddenAssembly(filterAssembly);
+
+                }
+                catch (Exception ex)
+                {
+                    InternalLogger.Trace(ex, "filtering Microsoft.Extensions.Logging.Filter failed. Not an issue probably");
+                }
+                
             }
-            catch (Exception)
+            catch (Exception ex)
             {
-                //ignore
+                InternalLogger.Debug(ex, "failure in ignoring assemblies. This could influence the ${callsite}");
             }
-
-            LogManager.AddHiddenAssembly(typeof(ConfigureExtensions).GetTypeInfo().Assembly);
         }
 
         /// <summary>