Fix panic related to connection state handling (versatica#70)

daily-co · Jun 10, 2021 · b8ceeb4 · b8ceeb4
1 parent bffd2d6
commit b8ceeb4
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/mediasoupclient/src/api/transport.rs b/mediasoupclient/src/api/transport.rs
@@ -57,11 +57,13 @@ impl From<ffi::transport::ConnectionState> for PeerConnectionState {
     }
 
     #[cfg(not(target_arch = "wasm32"))]
+    // In native, we need to look directly at the ICE connection state change it seems?
+    // Similar mapping to https://github.com/versatica/mediasoup-client/blob/a813c39bde5c9493436d2f6c7af7fa310a96decd/src/handlers/Firefox60.ts#L213
     fn from(s: ffi::transport::ConnectionState) -> Self {
         match s.0.as_str() {
             "new" => PeerConnectionState::New,
-            "connecting" => PeerConnectionState::Connecting,
-            "connected" => PeerConnectionState::Connected,
+            "checking" => PeerConnectionState::Connecting,
+            "connected" | "completed" => PeerConnectionState::Connected,
             "failed" => PeerConnectionState::Failed,
             "disconnected" => PeerConnectionState::Disconnected,
             "closed" => PeerConnectionState::Closed,