Merge pull request #102 from coenm/feature/threadsafety

Make FakeEnvVars ConcurrentDictionary
tonerdo · Aug 20, 2024 · dbf6aba · dbf6aba
2 parents 7a37f0d + abf5c3e
commit dbf6aba
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/src/DotNetEnv/Env.cs b/src/DotNetEnv/Env.cs
@@ -1,4 +1,5 @@
 using System;
+using System.Collections.Concurrent;
 using System.Linq;
 using System.Collections.Generic;
 using System.Globalization;
@@ -10,7 +11,7 @@ public class Env
     {
         public const string DEFAULT_ENVFILENAME = ".env";
 
-        public static Dictionary<string, string> FakeEnvVars = new Dictionary<string, string>();
+        public static ConcurrentDictionary<string, string> FakeEnvVars = new ConcurrentDictionary<string, string>();
 
         public static IEnumerable<KeyValuePair<string, string>> LoadMulti (string[] paths, LoadOptions options = null)
         {

diff --git a/src/DotNetEnv/Parsers.cs b/src/DotNetEnv/Parsers.cs
@@ -16,10 +16,7 @@ public static KeyValuePair<string, string> SetEnvVar (KeyValuePair<string, strin
 
         public static KeyValuePair<string, string> DoNotSetEnvVar (KeyValuePair<string, string> kvp)
         {
-            if (Env.FakeEnvVars.ContainsKey(kvp.Key)) {
-                Env.FakeEnvVars.Remove(kvp.Key);
-            }
-            Env.FakeEnvVars.Add(kvp.Key, kvp.Value);
+            Env.FakeEnvVars.AddOrUpdate(kvp.Key, kvp.Value, (_, v) => v);
             return kvp;
         }