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

Propose which modifications can be made in SDP (Section 6) #8

Closed
ekr opened this issue May 8, 2014 · 3 comments
Closed

Propose which modifications can be made in SDP (Section 6) #8

ekr opened this issue May 8, 2014 · 3 comments

Comments

@ekr
Copy link
Contributor

ekr commented May 8, 2014

Section 6 indicates that programmers can edit the SDP and lists changes which can and cannot be made. It explicitly is preliminary and needs to be fleshed out.

@fluffy
Copy link
Contributor

fluffy commented May 19, 2014

There are three categories - must reject, must act on , and ones that the browser can do whatever it wants but if it is not acted on then an error is generated

@fluffy
Copy link
Contributor

fluffy commented May 19, 2014

General approach is most things are in the "browser can do what it wants case"

@martinthomson
Copy link
Contributor

Discussed in DC: this applies only to changes on the local side; the gap between createOffer/createAnswer and setLocalDescription.

Proposed path is to mandate one of two actions:

  • Accept the changes and apply them
  • Reject the changes outright

We will enumerate certain changes that fall into either category, but these lists will not be exhaustive. All unspecified changes will either be accepted or rejected, and that decision will be left to the browser, but they cannot be ignored.

Obviously, some types of changes will be ignored. For instance, reordering of a= lines might not be detected as a change. These might carry semantics (as a=group-id does), but we can't require perfect detection.

@ekr ekr added the PR Pending label May 29, 2014
juberti added a commit that referenced this issue May 30, 2014
Rewrite modifying SDP section. Fixes #8
fluffy pushed a commit that referenced this issue Jan 29, 2016
Text around an pool of ICE Candiates
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

4 participants