[APT-10393] Prevent crashing if content does not load

Adds a catch block to content loading - this area isn't caught by things in the normal Reducer patterns that happens in playback.

Armadillo/src/main/java/com/scribd/armadillo/playback/PlaybackEngine.kt

@@ -12,6 +12,7 @@ import com.scribd.armadillo.ArmadilloPlayerChoreographer
 import com.scribd.armadillo.Constants
 import com.scribd.armadillo.Milliseconds
 import com.scribd.armadillo.StateStore
+import com.scribd.armadillo.actions.ErrorAction
 import com.scribd.armadillo.actions.FastForwardAction
 import com.scribd.armadillo.actions.NewAudioPlayableAction
 import com.scribd.armadillo.actions.PlaybackEngineReady
@@ -24,7 +25,9 @@ import com.scribd.armadillo.actions.SkipNextAction
 import com.scribd.armadillo.actions.SkipPrevAction
 import com.scribd.armadillo.di.Injector
 import com.scribd.armadillo.encryption.ExoplayerEncryption
+import com.scribd.armadillo.error.ArmadilloException
 import com.scribd.armadillo.error.MissingDataException
+import com.scribd.armadillo.error.PlaybackStartFailureException
 import com.scribd.armadillo.models.AudioPlayable
 import com.scribd.armadillo.models.Chapter
 import com.scribd.armadillo.models.PlaybackState
@@ -137,16 +140,21 @@ internal class ExoplayerPlaybackEngine(private var audioPlayable: AudioPlayable)
 
         exoPlayer = createExoplayerInstance(context, audioAttributes.exoPlayerAttrs, loadControl)
 
-        val mediaSource = mediaSourceRetriever.generateMediaSource(audioPlayable.request, context)
-        exoPlayer.setMediaSource(mediaSource)
-        exoPlayer.prepare()
+        try {
+            val mediaSource = mediaSourceRetriever.generateMediaSource(audioPlayable.request, context)
+            exoPlayer.setMediaSource(mediaSource)
+            exoPlayer.prepare()
 
-        exoPlayer.addListener(playerEventListener)
+            exoPlayer.addListener(playerEventListener)
 
-        exoPlayer.playWhenReady = config.isAutoPlay
+            exoPlayer.playWhenReady = config.isAutoPlay
 
-        stateModifier.dispatch(PlaybackEngineReady(true))
-        stateModifier.dispatch(PlayerStateAction(PlaybackState.PAUSED))
+            stateModifier.dispatch(PlaybackEngineReady(true))
+            stateModifier.dispatch(PlayerStateAction(PlaybackState.PAUSED))
+        } catch (ex: Exception) {
+            val armadilloException = if(ex is ArmadilloException) ex else PlaybackStartFailureException(cause = ex)
+            stateModifier.dispatch(ErrorAction(armadilloException))
+        }
     }
 
     override fun updateMediaRequest(mediaRequest: AudioPlayable.MediaRequest) {

RELEASE.md

@@ -1,5 +1,8 @@
 # Project Armadillo Release Notes
 
+## 1.6.1 
+- Prevents fatal crashing when content fails to load
+
 ## 1.6.0
 - Encrypts widevine drm keys. The Android minSDK has been changed to version 23 in order to support this feature.
 

gradle.properties

@@ -13,7 +13,7 @@ org.gradle.jvmargs=-Xmx1536m
 # org.gradle.parallel=true
 PACKAGE_NAME=com.scribd.armadillo
 GRADLE_PLUGIN_VERSION=7.2.0
-LIBRARY_VERSION=1.6.0
+LIBRARY_VERSION=1.6.1
 EXOPLAYER_VERSION=2.19.1
 RXJAVA_VERSION=2.2.4
 RXANDROID_VERSION=2.0.1