miosakuma
commented
7 months ago iOS SDK での利用は以下です。
- SDKInfo.versionString
- シグナリング connnect メッセージにおける
sora_client - HTTP Proxy の agent に設定するデフォルト文字列
- シグナリング connnect メッセージにおける
- WebRTCInfo.versionString
- シグナリング connnect メッセージにおける
libwebrtc
- シグナリング connnect メッセージにおける
大きく 2 点の修正を行なっています。
1. バージョン文字列を SDKInfo, WebRTCInfo で修正する
これまで Configration.swift, PeerChannel.swift などで SDK バージョン情報や、libwebrtc 情報を編集していましたが、これを SDKInfo, WebRTCInfo で行うように修正しました。バージョン情報は SDKInfo, WebRTCInfo 内で管理した方がよいと考えたためです。
コメントあればお願いしたいこと
2. シグナリング connect で送信される libwebrtc の内容を Android にあわせる
当初は branch-head の値が送信されていないことから始まりましたが、 最終的には Android の同項目と出力内容を合わせる対応としています。 Sora C++ SDK も () 内に M がないのでそちらに合わせることにしました。
修正前 "libwebrtc": "Shiguredo-build M123 (M123.3.0 41b1493)"
修正後 "libwebrtc": "Shiguredo-build M123 (123.6312.3.0 41b1493)"
Sora Android SDK の出力内容 "libwebrtc": "Shiguredo-build M121 (121.6167.4.0 0f741da)"
コメントあればお願いしたいこと
確認した内容
This pull request primarily focuses on improving the versioning system and handling of version strings in the Sora iOS SDK. The changes include adding a branch-heads to the version string in the
connectmessage of signaling, updating the WebRTC version, and centralizing the version string editing inSDKInfoandWebRTCInfo. Additionally, theagentinConfiguration.swiftandsoraClientinPeerChannel.swiftare now using the updated version strings.Versioning system improvements:
CHANGES.md: Added a change log entry for adding branch-heads to the version string in theconnectmessage of signaling, updating the WebRTC version, and centralizing the version string editing inSDKInfoandWebRTCInfo.Codebase changes:
Sora/Configuration.swift: Updated theagentstring to useSDKInfo.versionStringinstead of concatenating "Sora iOS SDK" withSDKInfo.version.Sora/PackageInfo.swift: Added aversionStringtoSDKInfoandWebRTCInfothat includes the version and branch-heads. [1] [2]Sora/PeerChannel.swift: UpdatedsoraClientandwebRTCVersionto use the newversionStringfromSDKInfoandWebRTCInforespectively.