Bug in RoundTripBench() in 1.2.4

[Jormungandr]

I've found what I'm pretty sure is a bug in the RoundTripResponse processing in version 1.2.4 of the client. I was getting nothing but 0 millisecond round trips, which was a) strange and b) totally unlike what ping was telling me.

Turns out the following line in SysHandler.HandleRoundTripBench() was at fault:

Code: Select all

parameters.Add("elapsed", Convert.ToInt32(res.TotalSeconds)); 

Changing it to the following restored the expected behavior:

Code: Select all

parameters.Add("elapsed", Convert.ToInt32(res.TotalMilliseconds)); 

Thought you guys might like to know. And while it's annoying that this was broken, it's great that I had the library source code available so that I could fix the problem.

[ThomasLund]

Thanks for the bug report! Will include it in next fix.