From 89a0541574c230e1b3a78280f46f7c4c288ef692 Mon Sep 17 00:00:00 2001 From: Greyson Parrelli Date: Wed, 3 Sep 2025 13:17:27 -0400 Subject: [PATCH] Ensure we don't archive thumbnails or stories. --- .../securesms/jobs/ArchiveThumbnailUploadJob.kt | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/app/src/main/java/org/thoughtcrime/securesms/jobs/ArchiveThumbnailUploadJob.kt b/app/src/main/java/org/thoughtcrime/securesms/jobs/ArchiveThumbnailUploadJob.kt index 048822a7a6..414a062c39 100644 --- a/app/src/main/java/org/thoughtcrime/securesms/jobs/ArchiveThumbnailUploadJob.kt +++ b/app/src/main/java/org/thoughtcrime/securesms/jobs/ArchiveThumbnailUploadJob.kt @@ -98,8 +98,13 @@ class ArchiveThumbnailUploadJob private constructor( return Result.success() } + if (!MediaUtil.isImageOrVideoType(attachment.contentType)) { + Log.w(TAG, "$attachmentId isn't visual media (contentType = ${attachment.contentType}). Skipping.") + return Result.success() + } + if (attachment.quote) { - Log.w(TAG, "$attachmentId is a quote, skipping.") + Log.w(TAG, "$attachmentId is a quote. Skipping.") return Result.success() } @@ -108,6 +113,11 @@ class ArchiveThumbnailUploadJob private constructor( return Result.success() } + if (SignalDatabase.messages.isStory(attachment.mmsId)) { + Log.w(TAG, "$attachmentId is a story. Skipping.") + return Result.success() + } + // TODO [backups] Determine if we actually need to upload or are reusing a thumbnail from another attachment val thumbnailResult = generateThumbnailIfPossible(attachment)