Nov 19, 2025
diff --git a/src/NLog/Config/LoggingConfigurationParser.cs b/src/NLog/Config/LoggingConfigurationParser.cs
                rulesInsertPosition = rulesCollection.Count;
            }

            if (rulesElement.Values.Count > 0)
            {
                foreach (var rulesOption in rulesElement.Values)
                {
                    var configException = new NLogConfigurationException($"Unrecognized value '{rulesOption.Key}'='{rulesOption.Value}' in section '{rulesElement.Name}'");
                    if (MustThrowConfigException(configException))
                        throw configException;
                }
            }

            foreach (var childItem in rulesElement.ValidChildren)
            {
                var loggingRule = ParseRuleElement(childItem);
            }
        }

        private void ParseTargetsElement(ValidatedConfigurationElement targetsElement)
        {
            targetsElement.AssertName("targets", "appenders");

            bool asyncWrap = ParseBooleanValue("async", targetsElement.GetOptionalValue("async", "false") ?? string.Empty, false);
            if (targetsElement.Values.Count > (asyncWrap ? 1 : 0))
            {
                foreach (var targetsOption in targetsElement.Values)
                {
                    if (string.Equals(targetsOption.Key?.Trim(), "async", StringComparison.OrdinalIgnoreCase))
                        continue;

                    var configException = new NLogConfigurationException($"Unrecognized value '{targetsOption.Key}'='{targetsOption.Value}' in section '{targetsElement.Name}'");
                    if (MustThrowConfigException(configException))
                        throw configException;
                }
            }

            ValidatedConfigurationElement? defaultWrapperElement = null;
            Dictionary<string, ValidatedConfigurationElement>? typeNameToDefaultTargetParameters = null;
diff --git a/tests/NLog.UnitTests/Config/XmlConfigTests.cs b/tests/NLog.UnitTests/Config/XmlConfigTests.cs
            {
                // Arrange
                var xml = @"<nlog internalLogLevel='oops' globalThreshold='noooos'>
                        <targets>
                        <targets option='unknown'>
                            <target name='debug' type='Debug' layout='${message}' />
                        </targets>
                        <rules>
                        <rules bingo='tada'>
                            <logger name='*' minlevel='debug' appendto='debug' />
                         </rules>
                    </nlog>";

                var logFactory = new LogFactory();
                var config = XmlLoggingConfiguration.CreateFromXmlString(xml, logFactory);
                logFactory.Configuration = config;
                var logger = logFactory.GetLogger("InvalidInternalLogLevel_shouldNotBreakLogging");
                var logger = logFactory.GetLogger("InvalidNLogAttributeValues_shouldNotBreakLogging");

                // Act
                logger.Debug("message 1");
Original file line number	Diff line number	Diff line change
Expand Up		@@ -547,6 +547,16 @@
		rulesInsertPosition = rulesCollection.Count;
		}

		if (rulesElement.Values.Count > 0)
		{
		foreach (var rulesOption in rulesElement.Values)
		{
		var configException = new NLogConfigurationException($"Unrecognized value '{rulesOption.Key}'='{rulesOption.Value}' in section '{rulesElement.Name}'");
		if (MustThrowConfigException(configException))
		throw configException;
		}
		}

		foreach (var childItem in rulesElement.ValidChildren)
		{
		var loggingRule = ParseRuleElement(childItem);
Expand DownExpand Up		@@ -838,11 +848,23 @@
		}
		}

		private void ParseTargetsElement(ValidatedConfigurationElement targetsElement)
Check failure on line 851 in src/NLog/Config/LoggingConfigurationParser.cs View check run for this annotation SonarQubeCloud/ SonarCloud Code Analysis Refactor this method to reduce its Cognitive Complexity from 28 to the 20 allowed. `See more on https://sonarcloud.io/project/issues?id=nlog2&issues=AZqdUunQCnHg2yY8GF5V&open=AZqdUunQCnHg2yY8GF5V&pullRequest=6045`
		{
		targetsElement.AssertName("targets", "appenders");

		bool asyncWrap = ParseBooleanValue("async", targetsElement.GetOptionalValue("async", "false") ?? string.Empty, false);
		if (targetsElement.Values.Count > (asyncWrap ? 1 : 0))
		{
		foreach (var targetsOption in targetsElement.Values)
		{
		if (string.Equals(targetsOption.Key?.Trim(), "async", StringComparison.OrdinalIgnoreCase))
		continue;

		var configException = new NLogConfigurationException($"Unrecognized value '{targetsOption.Key}'='{targetsOption.Value}' in section '{targetsElement.Name}'");
		if (MustThrowConfigException(configException))
		throw configException;
		}
		}

		ValidatedConfigurationElement? defaultWrapperElement = null;
		Dictionary<string, ValidatedConfigurationElement>? typeNameToDefaultTargetParameters = null;
Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -201,18 +201,18 @@ public void InvalidNLogAttributeValues_shouldNotBreakLogging()
		{
		// Arrange
		var xml = @"<nlog internalLogLevel='oops' globalThreshold='noooos'>
		<targets>
		<targets option='unknown'>
		<target name='debug' type='Debug' layout='${message}' />
		</targets>
		<rules>
		<rules bingo='tada'>
		<logger name='*' minlevel='debug' appendto='debug' />
		</rules>
		</nlog>";

		var logFactory = new LogFactory();
		var config = XmlLoggingConfiguration.CreateFromXmlString(xml, logFactory);
		logFactory.Configuration = config;
		var logger = logFactory.GetLogger("InvalidInternalLogLevel_shouldNotBreakLogging");
		var logger = logFactory.GetLogger("InvalidNLogAttributeValues_shouldNotBreakLogging");

		// Act
		logger.Debug("message 1");
Expand Down