Merge pull request #4625 from vector-im/feature/dla/outgoing_pstn_cal…

…l_fails select answer should be used for outgoing calls.
element-hq · Dec 3, 2021 · 9f4f15a · 9f4f15a
2 parents 1aa5321 + 767ead0
commit 9f4f15a
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/changelog.d/4621.bugfix b/changelog.d/4621.bugfix
@@ -0,0 +1 @@
+Fix for outgoing voip call via sip bridge failing after 1 minute.
diff --git a/...dk-android/src/main/java/org/matrix/android/sdk/internal/session/call/model/MxCallImpl.kt b/...dk-android/src/main/java/org/matrix/android/sdk/internal/session/call/model/MxCallImpl.kt
@@ -203,8 +203,11 @@ internal class MxCallImpl(
 
     override fun selectAnswer() {
         Timber.tag(loggerTag.value).v("select answer $callId")
-        if (isOutgoing) return
-        state = CallState.Answering
+        if (!isOutgoing) return
+        // This is an outgoing call, select the remote client that answered.
+        if (state != CallState.Dialing) {
+            Timber.tag(loggerTag.value).w("Expected state is CallState.Dialing got $state.")
+        }
         CallSelectAnswerContent(
                 callId = callId,
                 partyId = ourPartyId,
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		Fix for outgoing voip call via sip bridge failing after 1 minute.