Merge pull request #297 from loki-project/photo-sending-bug

Fix Attachment Download Retrying

src/org/thoughtcrime/securesms/jobs/AttachmentDownloadJob.java

@@ -1,8 +1,9 @@
 package org.thoughtcrime.securesms.jobs;
 
+import android.text.TextUtils;
+
 import androidx.annotation.NonNull;
 import androidx.annotation.VisibleForTesting;
-import android.text.TextUtils;
 
 import org.greenrobot.eventbus.EventBus;
 import org.thoughtcrime.securesms.ApplicationContext;
@@ -39,7 +40,7 @@ public class AttachmentDownloadJob extends BaseJob implements InjectableType {
 
   public static final String KEY = "AttachmentDownloadJob";
 
-  private static final int    MAX_ATTACHMENT_SIZE = 150 * 1024  * 1024;
+  private static final int    MAX_ATTACHMENT_SIZE  = 10 * 1024  * 1024;
   private static final String TAG                  = AttachmentDownloadJob.class.getSimpleName();
 
   private static final String KEY_MESSAGE_ID    = "message_id";
@@ -58,7 +59,7 @@ public class AttachmentDownloadJob extends BaseJob implements InjectableType {
     this(new Job.Parameters.Builder()
                            .setQueue("AttachmentDownloadJob" + attachmentId.getRowId() + "-" + attachmentId.getUniqueId())
                            .addConstraint(NetworkConstraint.KEY)
-                           .setMaxAttempts(25)
+                           .setMaxAttempts(5)
                            .build(),
          messageId,
          attachmentId,
@@ -183,19 +184,19 @@ public class AttachmentDownloadJob extends BaseJob implements InjectableType {
   SignalServiceAttachmentPointer createAttachmentPointer(Attachment attachment)
       throws InvalidPartException
   {
-    boolean isPublicAttachment = TextUtils.isEmpty(attachment.getKey()) && attachment.getDigest() == null;
+    boolean isOpenGroupContext = TextUtils.isEmpty(attachment.getKey()) && attachment.getDigest() == null;
 
     if (TextUtils.isEmpty(attachment.getLocation())) {
       throw new InvalidPartException("empty content id");
     }
 
-    if (TextUtils.isEmpty(attachment.getKey()) && !isPublicAttachment) {
+    if (TextUtils.isEmpty(attachment.getKey()) && !isOpenGroupContext) {
       throw new InvalidPartException("empty encrypted key");
     }
 
     try {
-      long   id    = Long.parseLong(attachment.getLocation());
-      if (isPublicAttachment) {
+      long id = Long.parseLong(attachment.getLocation());
+      if (isOpenGroupContext) {
         return new SignalServiceAttachmentPointer(id,
                                                   null,
                                                   new byte[0],
@@ -210,11 +211,6 @@ public class AttachmentDownloadJob extends BaseJob implements InjectableType {
       }
 
       byte[] key   = Base64.decode(attachment.getKey());
-      String relay = null;
-
-      if (TextUtils.isEmpty(attachment.getRelay())) {
-        relay = attachment.getRelay();
-      }
 
       if (attachment.getDigest() != null) {
         Log.i(TAG, "Downloading attachment with digest: " + Hex.toString(attachment.getDigest()));

src/org/thoughtcrime/securesms/util/AttachmentUtil.java

@@ -2,18 +2,17 @@ package org.thoughtcrime.securesms.util;
 
 
 import android.content.Context;
-import android.database.Cursor;
 import android.net.ConnectivityManager;
 import android.net.NetworkInfo;
+import android.text.TextUtils;
+
 import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
 import androidx.annotation.WorkerThread;
-import android.text.TextUtils;
 
 import org.thoughtcrime.securesms.attachments.AttachmentId;
 import org.thoughtcrime.securesms.attachments.DatabaseAttachment;
 import org.thoughtcrime.securesms.database.DatabaseFactory;
-import org.thoughtcrime.securesms.database.model.MessageRecord;
 import org.thoughtcrime.securesms.logging.Log;
 
 import java.util.Collections;
@@ -106,11 +105,6 @@ public class AttachmentUtil {
 
   @WorkerThread
   private static boolean isFromUnknownContact(@NonNull Context context, @NonNull DatabaseAttachment attachment) {
-    MessageRecord message;
-    try (Cursor messageCursor = DatabaseFactory.getMmsDatabase(context).getMessage(attachment.getMmsId())) {
-      message = DatabaseFactory.getMmsDatabase(context).readerFor(messageCursor).getNext();
-    }
-    if (message == null) { return true; }
     // We don't allow attachments to be sent unless we're friends with someone or the attachment is sent
     // in a group context. Auto-downloading attachments is therefore fine.
     return false;