Ensure the incoming call will not ring forever, in case the call is n…

…ot ended by another way (#8178) Add a safe 2 minutes timer.
element-hq · Oct 6, 2023 · 52082a9 · 52082a9
1 parent 9e74afc
commit 52082a9
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/changelog.d/8178.bugfix b/changelog.d/8178.bugfix
@@ -0,0 +1 @@
+Ensure the incoming call will not ring forever, in case the call is not ended by another way.
diff --git a/vector/src/main/java/im/vector/app/features/call/webrtc/WebRtcCallManager.kt b/vector/src/main/java/im/vector/app/features/call/webrtc/WebRtcCallManager.kt
@@ -34,6 +34,8 @@ import im.vector.app.features.call.vectorCallService
 import im.vector.app.features.session.coroutineScope
 import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.asCoroutineDispatcher
+import kotlinx.coroutines.delay
+import kotlinx.coroutines.launch
 import org.matrix.android.sdk.api.extensions.orFalse
 import org.matrix.android.sdk.api.extensions.tryOrNull
 import org.matrix.android.sdk.api.logger.LoggerTag
@@ -386,6 +388,14 @@ class WebRtcCallManager @Inject constructor(
                 // Maybe increase sync freq? but how to set back to default values?
             }
         }
+
+        // ensure the incoming call will not ring forever
+        sessionScope?.launch {
+            delay(2 * 60 * 1000 /* 2 minutes */)
+            if (mxCall.state is CallState.LocalRinging) {
+                onCallEnded(mxCall.callId, EndCallReason.INVITE_TIMEOUT, rejected = false)
+            }
+        }
     }
 
     override fun onCallAnswerReceived(callAnswerContent: CallAnswerContent) {
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		Ensure the incoming call will not ring forever, in case the call is not ended by another way.