Skip to content

Commit 6eed458

Browse files
committed
Use server timestamp for delivery duration metrics
1 parent 54fb0a6 commit 6eed458

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

service/src/main/java/org/whispersystems/textsecuregcm/controllers/MessageController.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -784,7 +784,7 @@ public CompletableFuture<Response> removePendingMessage(@ReadOnly @Auth Authenti
784784
.thenAccept(maybeDeletedMessage -> {
785785
maybeDeletedMessage.ifPresent(deletedMessage -> {
786786

787-
WebSocketConnection.recordMessageDeliveryDuration(deletedMessage.getTimestamp(),
787+
WebSocketConnection.recordMessageDeliveryDuration(deletedMessage.getServerTimestamp(),
788788
auth.getAuthenticatedDevice());
789789

790790
if (deletedMessage.hasSourceUuid() && deletedMessage.getType() != Type.SERVER_DELIVERY_RECEIPT) {

service/src/main/java/org/whispersystems/textsecuregcm/websocket/WebSocketConnection.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -237,7 +237,7 @@ private CompletableFuture<Void> sendMessage(final Envelope message, StoredMessag
237237
.thenApply(ignored -> null);
238238

239239
if (message.getType() != Envelope.Type.SERVER_DELIVERY_RECEIPT) {
240-
recordMessageDeliveryDuration(message.getTimestamp(), device);
240+
recordMessageDeliveryDuration(message.getServerTimestamp(), device);
241241
sendDeliveryReceiptFor(message);
242242
}
243243
} else {

0 commit comments

Comments
 (0)