Adds support for NLog to application logging

[jaffinito]

Description

• Adds support got NLog version 4 and 5 for application logging - metrics, forwarding, and local decorating
• Updated Microsoft.Extensions.Logging support to exclude NLog from decorating since NLog support automatic decorating of local logs
• Includes NLog tests for all logging integration tests on several versions
• MSI changes to include NLog

Author Checklist

• Unit tests, Integration tests, and Unbounded tests completed
• Performance testing completed with satisfactory results (if required)
• Agent Changelog or Lambda Agent changelog updated

Reviewer Checklist

• Perform code review
• Pull request was adequately tested (new/existing tests, performance tests)
• Review Changelog

[JcolemanNR]

Reviewed in pairing, and this looks ready to ship!

[snakefoot]

Maybe use isolated NLog.LogFactory like this:

var logFactory = new NLog.LogFactory();
var logConfig  = new NLog.Config.LoggingConfiguration();
var logConsole = new ConsoleTarget();
if (layoutOverride != null)
      logConsole = layoutOverride;
logConfig.AddTarget("console", logConsole);
logConfig.LoggingRules.Add(new NLog.Config.LoggingRule("*", NLog.LogLevel.Trace, logConsole));
logFactory.Configuration = logConfig;
return logFactory.GetLogger("LoggingTest");

Then ConfigureJsonLayoutAppenderForDecoration() can provide its JsonLayout as layoutOverride.

[jaffinito]

Thank you for this! I updated the adapter to setup the log this and all the tests passed!

[michaelgoin]

Looks pretty solid to me. Some minor comments that don't need to be addressed.

Would recommend the testing on the actual 5.0 version in this or a follow-up PR.

I did notice we access an internal "Message" property taht seems like we are assuming can never be null. Which may be the case but seems maybe slightly less guaranteed than a "FormattedMessage" without looking at the code. Just calling out to make sure that was known given we do string appending.