Merge pull request #261 from geekyed/5.x

Fixing an issue caused by the usage of a non thread safe Dictionary.
Vonage · Nov 23, 2021 · da8a1f0 · da8a1f0
2 parents dfa8be7 + 2558b4b
commit da8a1f0
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/Vonage/Logger/LogProvider.cs b/Vonage/Logger/LogProvider.cs
@@ -1,12 +1,14 @@
 using Microsoft.Extensions.Logging;
 using Microsoft.Extensions.Logging.Abstractions;
+using System.Collections.Concurrent;
 using System.Collections.Generic;
 using System;
+
 namespace Vonage.Logger
 {
     public class LogProvider
     {
-        private static IDictionary<string, ILogger> _loggers = new Dictionary<string, ILogger>();
+        private static IDictionary<string, ILogger> _loggers = new ConcurrentDictionary<string, ILogger>();
         private static ILoggerFactory _loggerFactory = new LoggerFactory();
 
         public static void SetLogFactory(ILoggerFactory factory)
@@ -20,7 +22,7 @@ public static ILogger GetLogger(string category)
         {
             if (!_loggers.ContainsKey(category))
             {
-                _loggers[category] = _loggerFactory?.CreateLogger(category)?? NullLogger.Instance;
+                _loggers[category] = _loggerFactory?.CreateLogger(category) ?? NullLogger.Instance;
             }
             return _loggers[category];
         }