Skip to content
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

Confusion on currentRemoteDescription.sdp need not match remoteDescription.sdp #1228

Closed
soareschen opened this issue May 19, 2017 · 2 comments

Comments

@soareschen
Copy link
Contributor

I was reading #921 and found #1004 that added:

Note that the value of currentRemoteDescription.sdp need not match the value of remoteDescription.sdp.

When I read the sentence I interpreted it as they may be different because localDescription may point to pendingLocalDescription instead. Didn't expect that to be related to #921.

@aboba
Copy link
Contributor

aboba commented Jul 18, 2017

Is there a proposed change that would make this more clear?

@soareschen
Copy link
Contributor Author

I think the note applies to both pending and current descriptions. It can be added as a general note under the attributes section as:

  • Note that the string value of sdp in pendingRemoteDescription, currentRemoteDescription, pendingLocalDescription, and currentLocalDescription need not be identical to the description.sdp value passed to the corresponding setRemoteDescription or setLocalDescription calls. e.g. the SDP lines can be reordered and modified.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants