Merge pull request #226 from shiguredo/feature/no-multistream-modific…

…ation

設定に応じて multistreamEnabled の値を更新する処理を削除する

CHANGES.md

@@ -11,6 +11,12 @@
 
 ## develop
 
+- [CHANGE] connect メッセージの `multistream` を true 固定で送信する処理を削除する破壊的変更
+  - Configration.role に .sendrecv を指定している場合に multistream を true に更新する処理を削除
+  - Configration.spotlightEnabled に .enabled を指定している場合に multistream を true に更新する処理を削除
+  - 結果、connect メッセージには Configration.multistreamEnabled に指定した値が送信される
+  - 今後は Configration.role に .sendrecv を指定している場合または Configration.spotlightEnabled に .enabled を指定している場合に Confgration.multistreamEnabled に false を指定すると接続エラーになる 
+  - @zztkm
 - [UPDATE] multistreamEnabled を非推奨扱いにする
   - multistreamEnabled の設定が不要なイニシャライザを `Configuration` に追加する
   - ドキュメントコメントに非推奨扱いの旨を追加する

Sora/PeerChannel.swift

@@ -294,22 +294,16 @@ class PeerChannel: NSObject, RTCPeerConnectionDelegate {
                      message: "did connect to signaling channel")
 
         var role: SignalingRole
-        var multistream = configuration.multistreamEnabled
-        if configuration.spotlightEnabled == .enabled {
-            multistream = true
-        }
         switch configuration.role {
         case .sendonly:
             role = .sendonly
         case .recvonly:
             role = .recvonly
         case .sendrecv:
             role = .sendrecv
-            multistream = true
         }
 
         let soraClient = "Sora iOS SDK \(SDKInfo.version)"
-
         let webRTCVersion = "Shiguredo-build \(WebRTCInfo.version) (\(WebRTCInfo.version.dropFirst()).\(WebRTCInfo.branch).\(WebRTCInfo.commitPosition).\(WebRTCInfo.maintenanceVersion) \(WebRTCInfo.shortRevision))"
 
         let simulcast = configuration.simulcastEnabled
@@ -321,7 +315,7 @@ class PeerChannel: NSObject, RTCPeerConnectionDelegate {
             metadata: configuration.signalingConnectMetadata,
             notifyMetadata: configuration.signalingConnectNotifyMetadata,
             sdp: sdp,
-            multistreamEnabled: multistream,
+            multistreamEnabled: configuration.multistreamEnabled,
             videoEnabled: configuration.videoEnabled,
             videoCodec: configuration.videoCodec,
             videoBitRate: configuration.videoBitRate,