Consider using a format string for TimeSpan.TotalMilliseconds in log messages

[Tragetaschen]

Describe the bug

Since .NET Core 3, floating-point value's ToString behavior has changed. I now frequently see things like 200.19760000000002 ms in the log which looks rather clumsy.

To Reproduce

var ts = TimeSpan.FromTicks(2001976);
Console.WriteLine(FormattableString.Invariant($"{ts.TotalMilliseconds} ms"));
Console.WriteLine(FormattableString.Invariant($"{ts.TotalMilliseconds:F4} ms"));

200.19760000000002 ms
200.1976 ms

Expected behavior

TimeSpan.TicksPerMilliSecond == 10000, so a format string of F4 shows all the relevant digits.

We've moved this issue to the Backlog milestone. This means that it is not going to be worked on for the coming release. We will reassess the backlog following the current release and consider this item at that time. To learn more about our issue management process and to have better expectation regarding different types of issues you can read our Triage Process.

[JunTaoLuo]

This seems like a simple clean up item, we'll look for contributors to help out. PRs are welcome!

[RonaldMariah]

Happy to contribute and create a PR for this. Any ideas/recommendations on the approach?

[davidfowl]

I thought @benaadams fixed this

[benaadams]

Yeah is 4 decimal digits after "Betterize Hosting Log" #10102

[davidfowl]

Yep! Closed as fixed