livekit / rust-sdks

LiveKit realtime and server SDKs for Rust
https://livekit.io
Apache License 2.0
204 stars 50 forks source link

Update protocol to v15 #394

Closed nbsp closed 2 months ago

nbsp commented 2 months ago

exported as LocalTrackSubscribed in the client and FFI.

at the moment the SFU assumes Rust doesn't accept TrackSubscribed so it doesn't send it: see https://github.com/livekit/livekit/pull/2939

davidzhao commented 2 months ago

Currently Rust stack is on protocol version 9. part of this update will require bringing it to 10+. and I think we should just bring it up to date to support protocol 15.

Here are the changes required (JS PRs)

nbsp commented 2 months ago

v10 through v13 added or already there. re: RequestResponse: we don't keep a list of pending requests and we don't care about arbitrary responses at all -- most of our signalclient requests go to ``. should i add it regardless and make it no-op?

davidzhao commented 2 months ago

v10 through v13 added or already there. re: RequestResponse: we don't keep a list of pending requests and we don't care about arbitrary responses at all -- most of our signalclient requests go to ``. should i add it regardless and make it no-op?

yeah no-op is fine. we can add the capability later of firing error responses for things, but not required to be protocol-compatible.