Start time doesn't support ms.

[acollard]

This makes is pretty much useless when comparing telemetry events.

The ISO string returned from this utility is only accurate to the second.

ApplicationInsights-PHP/ApplicationInsights/Channel/Telemetry_Channel.php

Line 161 in 95407ee

$envelope->setTime(Contracts\Utils::returnISOStringForTime($startTime));

And all start times should use microtime();

ApplicationInsights-PHP/ApplicationInsights/Channel/Telemetry_Channel.php

Line 158 in 95407ee

$startTime = $data->getTime();

Actual Behavior

Start times are truncated at seconds.

Expected Behavior

All times should be accurate to at least milliseconds.

Workaround

We've had to implement a workaround right before flushing. We store the start_time in microseconds, then rewrite the start time right before flushing.

[acollard]

I will try to put together a PR with the fixes.

This will require some breaking changes as all start times will now need to be in milliseconds.

[acollard]

This shouldn't break any code as passing in a time() will just ignore the milliseconds.