Merge pull request #305 from hotwired/remote-path-config-exceptions

Catch all exceptions when attempting to fetch the remote config and log errors
hotwired · Feb 23, 2024 · 95de8a7 · 95de8a7
2 parents c548547 + f568c23
commit 95de8a7
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/turbo/src/main/kotlin/dev/hotwire/turbo/config/TurboPathConfigurationRepository.kt b/turbo/src/main/kotlin/dev/hotwire/turbo/config/TurboPathConfigurationRepository.kt
@@ -5,6 +5,7 @@ import android.content.SharedPreferences
 import androidx.core.content.edit
 import dev.hotwire.turbo.http.TurboHttpClient
 import dev.hotwire.turbo.util.dispatcherProvider
+import dev.hotwire.turbo.util.logError
 import dev.hotwire.turbo.util.toJson
 import kotlinx.coroutines.withContext
 import okhttp3.Request
@@ -43,10 +44,15 @@ internal class TurboPathConfigurationRepository {
                 if (response.isSuccessful) {
                     response.body?.string()
                 } else {
+                    logError(
+                        "remotePathConfigurationFailure",
+                        Exception("location: ${request.url}, status code: ${response.code}")
+                    )
                     null
                 }
             }
-        } catch (e: IOException) {
+        } catch (e: Exception) {
+            logError("remotePathConfigurationException", e)
             null
         }
     }