credits for connamara#258

NEXT_VERSION.md

@@ -20,4 +20,5 @@ Changes since the last version:
 
 * (minor) #247 - support for CheckLatency/MaxLatency config settings (ruaanviljoen)
 * (minor) #256 - bug in 247, defaults override config (akamyshanov)
+* (minor) #258 - threadsafe SessionState.Get/Set to fix garbled message issue (jacsuper)
 

UnitTests/SessionStateTest.cs

@@ -22,8 +22,6 @@ public void TimedOut()
             System.DateTime now = System.DateTime.UtcNow;
             System.DateTime lastReceivedTime = now;
 
-
-
             now = now.AddMilliseconds(heartBtIntMillis);
             Assert.False(SessionState.TimedOut(now, heartBtIntMillis, lastReceivedTime));
             now = now.AddMilliseconds(heartBtIntMillis);
@@ -211,14 +209,12 @@ public void ThreadSafeSetAndGet() {
             //wait till done and assert results
             Assert.True(setEvent.WaitOne(10000), "Get or Set hung/timed out during concurrent usage");
             Assert.True(getEvent.WaitOne(10000), "Get or Set hung/timed out during concurrent usage");
-            Assert.AreEqual(setTable, getTable, "Garbled data read  in concurrent set and get (like between resendrequest and send)");
+            Assert.AreEqual(setTable, getTable, "Garbled data read in concurrent set and get (like between resendrequest and send)");
             Assert.AreEqual(errorsTable.Count, 0, "IOException occured in concurrent set and get (like between resendrequest and send)");
 
             //Tear down filestore
             state.Dispose();
             store.Dispose();
-
         }
-
     }
 }

web/views/about/credits.md

@@ -42,5 +42,6 @@ Contributors
 - Daniel Svensson
 - Ruann Viljoen
 - Andrey Kamyshanov
+- jacsuper