fix: use ArgumentNullException for null logManager in EthStatsClient (#10018)

PivasDesant · web-flow · commit 81f7afb473eb · 2025-12-24T01:59:55.000+01:00
Update EthStatsClient.cs
diff --git a/src/Nethermind/Nethermind.EthStats/Clients/EthStatsClient.cs b/src/Nethermind/Nethermind.EthStats/Clients/EthStatsClient.cs
@@ -32,7 +32,7 @@ public EthStatsClient(
             _urlFromConfig = urlFromConfig ?? throw new ArgumentNullException(nameof(urlFromConfig));
             _reconnectionInterval = reconnectionInterval;
             _messageSender = messageSender ?? throw new ArgumentNullException(nameof(messageSender));
-            _logger = logManager?.GetClassLogger() ?? throw new ArgumentException(nameof(logManager));
+            _logger = logManager?.GetClassLogger() ?? throw new ArgumentNullException(nameof(logManager));
         }
 
         internal string BuildUrl()

Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ public EthStatsClient(`
`32`	`32`	`_urlFromConfig = urlFromConfig ?? throw new ArgumentNullException(nameof(urlFromConfig));`
`33`	`33`	`_reconnectionInterval = reconnectionInterval;`
`34`	`34`	`_messageSender = messageSender ?? throw new ArgumentNullException(nameof(messageSender));`
`35`		`- _logger = logManager?.GetClassLogger() ?? throw new ArgumentException(nameof(logManager));`
	`35`	`+ _logger = logManager?.GetClassLogger() ?? throw new ArgumentNullException(nameof(logManager));`
`36`	`36`	`}`
`37`	`37`
`38`	`38`	`internal string BuildUrl()`