-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
RTC::Consumer::GetStats crash #185
Comments
Thanks. Those logs look like your app's custom logs. May you please set full logging in mediasoup please?: https://mediasoup.org/documentation/mediasoup/debugging/ Also, could you affirm that the crash happens when closing the Consumer (due to sender or receiver peer closure)? |
@jmillan let's have this issue in the radar. We may have to redesign the profiles design since we are having many problems like this one. I think we should also pass the "default" profile from the Producer to the Consumer rather than assuming that, if no profiles have been set, then the default is used. Just wondering. |
Here are the mediasoup logs. I'm working on some client logic at the moment for reconnects to the sfu, and this is happening pretty consistently when I kill the sfu process, restart it, and the client auto-reconnects.
|
So the answer to your prior question I think is, no, I now don't think that the crash is happening when closing the consumer, because in the case above the consumer was never closed. |
@ibc, I definitely agree with the fact that we are assuming things here.
Yeah, Will check it as soon as possible. |
I'm tending to run into this more and more. Is there a workaround that could alleviate the issue for me? My understanding of what @jmillan mentioned is that getStats is being called before Consumer internally resolves its effectiveProfile. Would waiting until some event occurs to query for stats solve the issue in the meantime? |
Hi, I'll look into this today. You may for now just wait some time since consumer creation to requesting its stats. After this is fixed, the consumer's profile will tell whether there are any stats to retrieve: Ie: 'NONE: no RTP is being sent to the consumer' |
May you please try the latest master revision? |
Seems good so far. I'll let you know if I encounter it again. |
Thanks for reporting! |
Crashing with
on mediasoup v2.0.10 in OSX.
I don't know if this is relevant based on the error, but the getStats implementation I have is polling for stats:
I don't know if there could be a race internally in mediasoup between the consumer closing and getStats being called with such an implementation.
Also, don't know if it helps but here are some sfu logs that come before it:
The text was updated successfully, but these errors were encountered: