diff --git a/app/src/main/java/org/thoughtcrime/securesms/jobs/MultiDeviceMessageRequestResponseJob.java b/app/src/main/java/org/thoughtcrime/securesms/jobs/MultiDeviceMessageRequestResponseJob.java index b698e17f3..0a206fc0b 100644 --- a/app/src/main/java/org/thoughtcrime/securesms/jobs/MultiDeviceMessageRequestResponseJob.java +++ b/app/src/main/java/org/thoughtcrime/securesms/jobs/MultiDeviceMessageRequestResponseJob.java @@ -101,8 +101,8 @@ public class MultiDeviceMessageRequestResponseJob extends BaseJob { SignalServiceMessageSender messageSender = ApplicationDependencies.getSignalServiceMessageSender(); Recipient recipient = Recipient.resolved(threadRecipient); - if (!recipient.hasServiceId()) { - Log.i(TAG, "Queued for recipient without ServiceId"); + if (!recipient.isGroup() && !recipient.hasServiceId()) { + Log.i(TAG, "Queued for non-group recipient without ServiceId"); return; } diff --git a/app/src/main/java/org/thoughtcrime/securesms/messages/MessageContentProcessor.java b/app/src/main/java/org/thoughtcrime/securesms/messages/MessageContentProcessor.java index 65c7eac4f..d51de77b7 100644 --- a/app/src/main/java/org/thoughtcrime/securesms/messages/MessageContentProcessor.java +++ b/app/src/main/java/org/thoughtcrime/securesms/messages/MessageContentProcessor.java @@ -1102,7 +1102,7 @@ public final class MessageContentProcessor { if (response.getPerson().isPresent()) { recipient = Recipient.externalPush(response.getPerson().get()); } else if (response.getGroupId().isPresent()) { - GroupId groupId = GroupId.v1(response.getGroupId().get()); + GroupId groupId = GroupId.push(response.getGroupId().get()); recipient = Recipient.externalPossiblyMigratedGroup(groupId); } else { warn("Message request response was missing a thread recipient! Skipping.");