fix(events): stuck on processing calling events (#2145)

* fix(events): stuck on processing calling events (#2144) * ci: trigger build --------- Co-authored-by: Vitor Hugo Schwaab <[email protected]>
wireapp · Oct 17, 2023 · 2a68af4 · 2a68af4
1 parent a3963e9
commit 2a68af4
Showing 1 changed file with 13 additions and 3 deletions.
diff --git a/...rc/commonJvmAndroid/kotlin/com/wire/kalium/logic/feature/call/scenario/OnConfigRequest.kt b/...rc/commonJvmAndroid/kotlin/com/wire/kalium/logic/feature/call/scenario/OnConfigRequest.kt
@@ -40,12 +40,22 @@ class OnConfigRequest(
         callingScope.launch {
             callRepository.getCallConfigResponse(limit = null)
                 .fold({
-                    callingLogger.i("[OnConfigRequest] - Error: $it")
-                    // TODO: Add a better way to handle the Core Failure?
+                    callingLogger.w("[OnConfigRequest] - Error: $it")
+                    // We can call config_update with an error if there was a connectivity issue
+                    // AVS will eventually ask us again for the config
+                    // TODO(improvement): We can retry it ourselves and improve the app responsiveness.
+                    //                    Maybe add a retry mechanism that listens for the network state
+                    //                    Caches the config string and exposes a "invalidate" function
+                    //                    That we could call when AVS requests new config.
+                    calling.wcall_config_update(
+                        inst = inst,
+                        error = 1,
+                        jsonString = ""
+                    )
                 }, { config ->
                     calling.wcall_config_update(
                         inst = inst,
-                        error = 0, // TODO(calling): http error from internal json
+                        error = 0,
                         jsonString = config
                     )
                     callingLogger.i("[OnConfigRequest] - wcall_config_update()")