Merge pull request #153 from shiguredo/feature/disconnect-listener-on…

…close-last

SoraMediaChannel.Listener.onCloseの呼び出しタイミングを修正する

CHANGES.md

@@ -25,6 +25,10 @@
   - @zztkm
 - [ADD] サイマルキャストの映像のエンコーディングパラメーター `scaleResolutionDownTo` を追加する
   - @zztkm
+- [FIX] `SoraMediaChannel.internalDisconnect` での `SoraMediaChannel.Listener.onClose` の呼び出しタイミングを切断処理がすべて完了したあとに修正する
+  - 切断処理が終了する前に `onClose` を呼び出していたため、切断処理が完了してから呼び出すように修正
+  - `contactSignalingEndpoint` と `connectedSignalingEndpoint` は onClose で参照される可能性があるため、onClose 実行よりあとに null になるように onClose に合わせて処理順を変更
+  - @zztkm
 
 ## 2025.1.0
 

sora-android-sdk/src/main/kotlin/jp/shiguredo/sora/sdk/channel/SoraMediaChannel.kt

@@ -957,13 +957,6 @@ class SoraMediaChannel @JvmOverloads constructor(
         }
         compositeDisposable.dispose()
 
-        listener?.onClose(this)
-        listener = null
-
-        // アプリケーションで定義された切断処理を実行した後に contactSignalingEndpoint と connectedSignalingEndpoint を null にする
-        contactSignalingEndpoint = null
-        connectedSignalingEndpoint = null
-
         // 既に type: disconnect を送信しているので、 disconnectReason は null で良い
         signaling?.disconnect(null)
         signaling = null
@@ -974,6 +967,13 @@ class SoraMediaChannel @JvmOverloads constructor(
         // 既に type: disconnect を送信しているので、 disconnectReason は null で良い
         peer?.disconnect(null)
         peer = null
+
+        listener?.onClose(this)
+        listener = null
+
+        // onClose によってアプリケーションで定義された切断処理を実行した後に contactSignalingEndpoint と connectedSignalingEndpoint を null にする
+        contactSignalingEndpoint = null
+        connectedSignalingEndpoint = null
     }
 
     private fun sendDisconnectOverWebSocket(disconnectReason: SoraDisconnectReason) {