-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Empty ID and StreamID when simulcasting #1808
Comments
I think I've seen similar cases (null ids) in our project. However we avoided using theses IDs directly in logic. |
Found the issue! Should have a fix soon. libwebrtc is able to handle a So we have two things to fix here.
|
Sean-Der
added a commit
that referenced
this issue
Aug 24, 2021
If we have a media section with no SSRC we would fire an OnTrack. This code now properly ignores a MediaSection that has a rid attribute. Resolves #1808
pionbot
pushed a commit
that referenced
this issue
Aug 24, 2021
If we have a media section with no SSRC we would fire an OnTrack. This code now properly ignores a MediaSection that has a rid attribute. Resolves #1808
Sean-Der
added a commit
that referenced
this issue
Aug 24, 2021
If we have a media section with no SSRC we would fire an OnTrack. This code now properly ignores a MediaSection that has a rid attribute. Resolves #1808
Confirmed, thanks for the fix. |
daonb
pushed a commit
to daonb/webrtc
that referenced
this issue
Nov 20, 2022
daonb
pushed a commit
to daonb/webrtc
that referenced
this issue
Nov 20, 2022
If we have a media section with no SSRC we would fire an OnTrack. This code now properly ignores a MediaSection that has a rid attribute. Resolves pion#1808
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Browser is offerer, and offers a PC with a single sendonly transceiver with two simulcast tracks. When Pion fires OnTrack, both track.StreamID() and track.ID() are empty.
This does not happen if either of the following is true:
The offer sent by the browser is correct as far as I can see (it contains a well-formed a=msid line in the simulcast m section).
The text was updated successfully, but these errors were encountered: