Skip to content

Commit 39f473b

Browse files
Merge pull request #412 from adnanyaqoobvirk/master
Fixes custom telemetry timestamp bug
2 parents 28dec6f + e9118ac commit 39f473b

File tree

2 files changed

+12
-1
lines changed

2 files changed

+12
-1
lines changed

core/src/main/java/com/microsoft/applicationinsights/TelemetryClient.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -394,7 +394,9 @@ public void track(Telemetry telemetry) {
394394
return;
395395
}
396396

397-
telemetry.setTimestamp(new Date());
397+
if (telemetry.getTimestamp() == null) {
398+
telemetry.setTimestamp(new Date());
399+
}
398400

399401
TelemetryContext ctx = this.getContext();
400402

core/src/test/java/com/microsoft/applicationinsights/TelemetryClientTests.java

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -381,6 +381,15 @@ public void testTrackPageViewWithPageViewTelemetry() {
381381
verifyAndGetLastEventSent();
382382
}
383383

384+
@Test
385+
public void testTrackWithCustomTelemetryTimestamp() {
386+
Date timestamp = new Date(10000);
387+
client.track(new RequestTelemetry("Name", timestamp, 1, "200", true));
388+
389+
Telemetry telemetry = verifyAndGetLastEventSent();
390+
assertEquals(telemetry.getTimestamp(), timestamp);
391+
}
392+
384393
@Test
385394
public void testTrack() {
386395
TraceTelemetry telemetry = new TraceTelemetry("test");

0 commit comments

Comments
 (0)