-
Notifications
You must be signed in to change notification settings - Fork 163
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Using codec unsupported by the server causes uncaught error in promise #42
Comments
hey @bekriebel, I'm unable to reproduce this error. Which browser / OS did you see this with? Can you reproduce it with the sample JS client? If so, please attach a code snippet on what you've changed. |
@davidzhao Yeah, I'm able to reproduce with the client as well. Tested with both Chrome and Edge on Windows 11 (I'll try again with Windows 10 a bit later, but I doubt that's an issue considering that the video works and it's just unmute that is failing). The only change needed to the JS client sample is to add client-sdk-js/example/sample.ts Lines 177 to 185 in ee44df2
This is connecting to a server that is v0.12.5 with my |
Ahh, I figured it out. I didn't realize that room:
enabled_codecs:
- mime: audio/opus
- mime: video/vp8
- mime: video/h264 to my livekit-config it is no longer throwing the error. Ideally this would output a warning or error message and fall back to VP8 when h264 isn't available. |
ah yes.. that would do it. we removed h264 from the default due to a limitation with supporting tracks using multiple codecs in the same room. Once that bug is fixed, we should be able to add both into enabled_codecs by default. for now, just a heads up that mixing different video codecs in the same room is likely to lead to bugs where participants wouldn't be able to receive certain tracks that's different from their initial subscription. |
Thanks. Yeah, I noticed that mixing the codecs wasn't working very well. I was also getting a lot of frame drops with h264, so I've just disabled it again for now. I updated the issue to reflect what is actually happening. Feel free to close if you consider it a non-issue, though. |
oh interesting.. I think we should keep it open for now. I was able to reproduce it with the same setup. |
Using v0.12.1 from nmpjs
videoCodec: "h264"
The text was updated successfully, but these errors were encountered: