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

Extension methods for RTCIceTransports from PeerConnection #14

Open
steveanton opened this issue Jun 12, 2018 · 3 comments
Open

Extension methods for RTCIceTransports from PeerConnection #14

steveanton opened this issue Jun 12, 2018 · 3 comments
Assignees
Labels

Comments

@steveanton
Copy link
Collaborator

How do the extension methods defined in this specification apply to the RTCIceTransports returned from PeerConnection (via RTCDtlsTransport)?

Possible ideas:

  • Throw an exception if the object is from a PeerConnection.
  • Do not include the methods in the prototype of the RTCIceTransport returned from RTCDtlsTransport (is this valid?).
  • Specify how the methods interact with the offer/answer model of PeerConnection.
@aboba
Copy link
Contributor

aboba commented Jun 12, 2018

I don't think it makes sense for RTCIceTransport extension methods like gather() or start() to be called on an RTCIceTransport vended by an RTCPeerConnection. So I would throw an exception.

@aboba aboba added the question label Jun 12, 2018
@aboba aboba added PR needed A pull request is needed. and removed question labels Jul 31, 2018
@aboba aboba added question and removed PR needed A pull request is needed. labels Jan 24, 2020
@aboba
Copy link
Contributor

aboba commented Jan 24, 2020

@steveanton
Copy link
Collaborator Author

Yeah, does appear to be the case.

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

No branches or pull requests

2 participants