fixed ConfigureHiddenAssemblies not working for .NET Core 2.0

[agentschmitt]

Fixed ConfigureHiddenAssemblies not working for .NET Core 2.0 Applications.
This solves issue: NLog/NLog#2647

[304NotModified]

Thanks,

Isn't this an issue with the obsoletes?

NLog.Extensions.Logging/src/NLog.Extensions.Logging/Extensions/ConfigureExtensions.cs

Lines 111 to 130 in baa7dd8

	[Obsolete("Instead use NLog.LogManager.LoadConfiguration()")]
	public static LoggingConfiguration ConfigureNLog(this ILoggerFactory loggerFactory, string configFileRelativePath)
	{
	ConfigureHiddenAssemblies();
	return LogManager.LoadConfiguration(configFileRelativePath).Configuration;
	}
	/// <summary>
	/// Apply NLog configuration from config object.
	/// </summary>
	/// <param name="loggerFactory"></param>
	/// <param name="config">New NLog config.</param>
	/// <returns>Current configuration for chaining.</returns>
	[Obsolete("Instead assign property NLog.LogManager.Configuration")]
	public static LoggingConfiguration ConfigureNLog(this ILoggerFactory loggerFactory, LoggingConfiguration config)
	{
	ConfigureHiddenAssemblies();
	LogManager.Configuration = config;
	return config;
	}

(the method is still not called in .NET core 2?)

[agentschmitt]

The Obsoletes are also not working, cause the code inside of ConfigureHiddenAssemblies() is filtered out with the pre compiler condition (#if NETCORE1_0 && !NET451). So it is missing for NETCORE2_0.

[snakefoot]

Have created #210

[agentschmitt]

I think #210 is a more complete solution for the whole Nlog project. So i will close mine 👍

[snakefoot]

@agentschmitt Nlog.Extensions.Logging 1.0.1 has now been released. You can test if it works for you

[agentschmitt]

@snakefoot yes thx it is working now for .NET Core Application with HostBuilder.