chore: improve logging for backups (WPB-12113) 🍒 (#3096)

* chore: improve logging for backups (WPB-12113) (#3093) * chore: improve logging for backups * chore: improve logging for backups * chore: detekt * chore: empty commit bump --------- Co-authored-by: Yamil Medina <[email protected]>
wireapp · Nov 7, 2024 · f417fd4 · f417fd4
1 parent bbe46d0
commit f417fd4
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 4 deletions.
diff --git a/logic/src/commonMain/kotlin/com/wire/kalium/logic/feature/backup/RestoreBackupUseCase.kt b/logic/src/commonMain/kotlin/com/wire/kalium/logic/feature/backup/RestoreBackupUseCase.kt
@@ -95,7 +95,13 @@ internal class RestoreBackupUseCaseImpl(
                         importEncryptedBackup(extractedBackupRootPath, password)
                     }
                 }
-                .fold({ it }, { RestoreBackupResult.Success })
+                .fold({ error ->
+                    kaliumLogger.e("$TAG Failed to restore the backup, reason: ${error.failure}")
+                    error
+                }, {
+                    kaliumLogger.i("$TAG Backup restored successfully")
+                    RestoreBackupResult.Success
+                })
         }
 
     private suspend fun importUnencryptedBackup(
@@ -140,7 +146,7 @@ internal class RestoreBackupUseCaseImpl(
         val extractedFilesRootPath = createExtractedFilesRootPath()
         return extractFiles(tempCompressedFileSource, extractedFilesRootPath)
             .fold({
-                kaliumLogger.e("Failed to extract backup files")
+                kaliumLogger.e("$TAG Failed to extract backup files")
                 Either.Left(Failure(BackupIOFailure("Failed to extract backup files")))
             }, {
                 Either.Right(extractedFilesRootPath)
@@ -176,7 +182,7 @@ internal class RestoreBackupUseCaseImpl(
         return if (backupSize > 0) {
             // On successful decryption, we still need to extract the zip file to do sanity checks and get the database file
             extractFiles(kaliumFileSystem.source(extractedBackupPath), extractedBackupRootPath).fold({
-                kaliumLogger.e("Failed to extract encrypted backup files")
+                kaliumLogger.e("$TAG Failed to extract encrypted backup files")
                 Either.Left(Failure(BackupIOFailure("Failed to extract encrypted backup files")))
             }, {
                 kaliumFileSystem.delete(extractedBackupPath)

diff --git a/logic/src/commonMain/kotlin/com/wire/kalium/logic/feature/backup/RestoreWebBackupUseCase.kt b/logic/src/commonMain/kotlin/com/wire/kalium/logic/feature/backup/RestoreWebBackupUseCase.kt
@@ -79,7 +79,13 @@ internal class RestoreWebBackupUseCaseImpl(
                 importWebBackup(backupRootPath, this)
             } else {
                 Either.Left(IncompatibleBackup("invoke: The provided backup format is not supported"))
-            }.fold({ RestoreBackupResult.Failure(it) }, { RestoreBackupResult.Success })
+            }.fold({ error ->
+                kaliumLogger.e("$TAG Failed to restore the backup, reason: $error")
+                RestoreBackupResult.Failure(error)
+            }, {
+                kaliumLogger.i("$TAG Successfully restored the backup")
+                RestoreBackupResult.Success
+            })
         }
 
     private suspend fun importWebBackup(