Merge pull request #5 from naile/master

Ensure concurrent instances don't run on the same port
poulfoged · Jun 13, 2017 · 1cc8e28 · 1cc8e28
2 parents 1a62f7a + 2bbd9de
commit 1cc8e28
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/source/RedisInside/Redis.cs b/source/RedisInside/Redis.cs
@@ -1,4 +1,5 @@
 using System;
+using System.Collections.Concurrent;
 using System.Diagnostics;
 using System.Net;
 using System.Text;
@@ -93,13 +94,19 @@ public void Dispose()
 
         private class Config : IConfig
         {
+            private static readonly ConcurrentDictionary<int, byte> usedPorts = new ConcurrentDictionary<int, byte>();
             private static readonly Random random = new Random();
             internal Action<string> logger;
             internal int port;
 
             public Config()
             {
-                port = random.Next(49152, 65535 + 1);
+                do
+                {
+                    port = random.Next(49152, 65535 + 1);
+                } while (usedPorts.ContainsKey(port));
+
+                usedPorts.AddOrUpdate(port, i => byte.MinValue, (i, b) => byte.MinValue);
                 logger = message => Trace.WriteLine(message);
             }