From 9bf6922d97acaddf61e65fe3611001a978d8cf89 Mon Sep 17 00:00:00 2001 From: Greyson Parrelli Date: Tue, 9 May 2023 12:12:29 -0400 Subject: [PATCH] Ensure users have a service identifier before sending receipts. --- .../thoughtcrime/securesms/jobs/SendDeliveryReceiptJob.java | 5 +++++ .../org/thoughtcrime/securesms/jobs/SendReadReceiptJob.java | 5 +++++ 2 files changed, 10 insertions(+) diff --git a/app/src/main/java/org/thoughtcrime/securesms/jobs/SendDeliveryReceiptJob.java b/app/src/main/java/org/thoughtcrime/securesms/jobs/SendDeliveryReceiptJob.java index 0797465aaa..91dce76d8f 100644 --- a/app/src/main/java/org/thoughtcrime/securesms/jobs/SendDeliveryReceiptJob.java +++ b/app/src/main/java/org/thoughtcrime/securesms/jobs/SendDeliveryReceiptJob.java @@ -112,6 +112,11 @@ public class SendDeliveryReceiptJob extends BaseJob { return; } + if (!recipient.hasServiceId() && !recipient.hasE164()) { + Log.w(TAG, "No serviceId or e164!"); + return; + } + SignalServiceAddress remoteAddress = RecipientUtil.toSignalServiceAddress(context, recipient); SignalServiceReceiptMessage receiptMessage = new SignalServiceReceiptMessage(SignalServiceReceiptMessage.Type.DELIVERY, Collections.singletonList(messageSentTimestamp), diff --git a/app/src/main/java/org/thoughtcrime/securesms/jobs/SendReadReceiptJob.java b/app/src/main/java/org/thoughtcrime/securesms/jobs/SendReadReceiptJob.java index 815709ec9d..54eaa59223 100644 --- a/app/src/main/java/org/thoughtcrime/securesms/jobs/SendReadReceiptJob.java +++ b/app/src/main/java/org/thoughtcrime/securesms/jobs/SendReadReceiptJob.java @@ -178,6 +178,11 @@ public class SendReadReceiptJob extends BaseJob { return; } + if (!recipient.hasServiceId() && !recipient.hasE164()) { + Log.w(TAG, "No serviceId or e164!"); + return; + } + SignalServiceMessageSender messageSender = ApplicationDependencies.getSignalServiceMessageSender(); SignalServiceAddress remoteAddress = RecipientUtil.toSignalServiceAddress(context, recipient); SignalServiceReceiptMessage receiptMessage = new SignalServiceReceiptMessage(SignalServiceReceiptMessage.Type.READ, messageSentTimestamps, timestamp);