fix: catch exception when parsing file for transcription (#42)

Author: dixidroid

core/src/main/java/org/openedx/core/module/TranscriptManager.kt

@@ -5,6 +5,7 @@ import okhttp3.OkHttpClient
 import org.openedx.core.module.download.AbstractDownloader
 import org.openedx.core.utils.Directories
 import org.openedx.core.utils.IOUtils
+import org.openedx.core.utils.Logger
 import org.openedx.core.utils.Sha1Util
 import org.openedx.foundation.utils.FileUtil
 import subtitleFile.FormatSRT
@@ -21,6 +22,8 @@ class TranscriptManager(
     val fileUtil: FileUtil
 ) {
 
+    private val logger = Logger(TAG)
+
     private val transcriptDownloader = object : AbstractDownloader() {
         override val client: OkHttpClient
             get() = OkHttpClient.Builder().build()
@@ -68,9 +71,11 @@ class TranscriptManager(
             )
             if (result == AbstractDownloader.DownloadResult.SUCCESS) {
                 getInputStream(downloadLink)?.let {
-                    val transcriptTimedTextObject =
-                        convertIntoTimedTextObject(it)
-                    transcriptObject = transcriptTimedTextObject
+                    try {
+                        transcriptObject = convertIntoTimedTextObject(it)
+                    } catch (e: NullPointerException) {
+                        logger.e(throwable = e, submitCrashReport = true)
+                    }
                 }
             }
         }
@@ -84,7 +89,7 @@ class TranscriptManager(
             try {
                 transcriptObject = convertIntoTimedTextObject(transcriptInputStream)
             } catch (e: Exception) {
-                e.printStackTrace()
+                logger.e(throwable = e, submitCrashReport = true)
             }
         } else {
             startTranscriptDownload(transcriptUrl)
@@ -128,4 +133,8 @@ class TranscriptManager(
         }
         return null
     }
+
+    private companion object {
+        const val TAG = "TranscriptManager"
+    }
 }

core/src/main/java/org/openedx/core/utils/Logger.kt

@@ -1,9 +1,16 @@
 package org.openedx.core.utils
 
 import android.util.Log
+import com.google.firebase.crashlytics.FirebaseCrashlytics
+import org.koin.core.component.KoinComponent
+import org.koin.core.component.inject
 import org.openedx.core.BuildConfig
+import org.openedx.core.config.Config
+
+class Logger(private val tag: String) : KoinComponent {
+
+    private val config by inject<Config>()
 
-class Logger(private val tag: String) {
     fun d(message: () -> String) {
         if (BuildConfig.DEBUG) Log.d(tag, message())
     }
@@ -12,6 +19,13 @@ class Logger(private val tag: String) {
         if (BuildConfig.DEBUG) Log.e(tag, message())
     }
 
+    fun e(throwable: Throwable, submitCrashReport: Boolean = false) {
+        if (BuildConfig.DEBUG) throwable.printStackTrace()
+        if (submitCrashReport && config.getFirebaseConfig().enabled) {
+            FirebaseCrashlytics.getInstance().recordException(throwable)
+        }
+    }
+
     fun i(message: () -> String) {
         if (BuildConfig.DEBUG) Log.i(tag, message())
     }