From 5cb47dae3539598af72bde53579343b22027d31e Mon Sep 17 00:00:00 2001 From: onurays Date: Wed, 25 Mar 2020 18:03:20 +0300 Subject: [PATCH] Return a failure message if the file cannot be opened. --- .../internal/session/content/UploadContentWorker.kt | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/content/UploadContentWorker.kt b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/content/UploadContentWorker.kt index 21ab649c23..1b736d349f 100644 --- a/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/content/UploadContentWorker.kt +++ b/matrix-sdk-android/src/main/java/im/vector/matrix/android/internal/session/content/UploadContentWorker.kt @@ -90,7 +90,14 @@ internal class UploadContentWorker(val context: Context, params: WorkerParameter var newImageAttributes: NewImageAttributes? = null try { - val inputStream = context.contentResolver.openInputStream(attachment.queryUri) ?: return Result.success() + val inputStream = context.contentResolver.openInputStream(attachment.queryUri) + ?: return Result.success( + WorkerParamsFactory.toData( + params.copy( + lastFailureMessage = "Cannot openInputStream for file: " + attachment.queryUri.toString() + ) + ) + ) inputStream.use { var uploadedThumbnailUrl: String? = null @@ -122,8 +129,7 @@ internal class UploadContentWorker(val context: Context, params: WorkerParameter uploadedThumbnailUrl = contentUploadResponse.contentUri } catch (t: Throwable) { - Timber.e(t) - return handleFailure(params, t) + Timber.e(t, "Thumbnail update failed") } }