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.

Sign up for GitHub

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

Jump to bottom

RTCIceTransport 'failed' state #199

Closed
robin-raymond opened this issue May 13, 2015 · 3 comments
Closed

RTCIceTransport 'failed' state #199

robin-raymond opened this issue May 13, 2015 · 3 comments
Labels
1.0 compatibility issue 1.1 PR exists

Comments

@robin-raymond
Copy link
Contributor

Spec does not have an RTCIceTransport state of failed which exists both in the ICE RFC and trickle ice. We do have a disconnected state.
@robin-raymond
Copy link
Contributor Author

disconnected

The RTCIceTransport has received at least one local and remote candidate, and a local and remote RTCIceCandidateComplete dictionary was not added as the last candidate but all appropriate candidate pairs thus far have been tested and failed (or prior obtained consent was removed). Other other candidate pairs may become available for testing as new candidates are trickled thus the transport is not failed yet.

failed

A local and remote RTCIceCandidateComplete dictionary was added as the last candidate to the RTCIceTransport and all appropriate candidate pairs have been tested and failed.

.
FYI - We could use merge checking and disconnected if we don't need distinction when it's still in the checking process vs all checks possible have completed thus far and failed...

@aboba
Copy link
Contributor

aboba commented May 19, 2015

Here is a potential new state diagram:
icestate-new

@aboba
Copy link
Contributor

aboba commented Jun 3, 2015

Here is a revision to the state diagram. Since it is possible to call iceTransport.start() again with a new IceGatherer argument, transitions to "New" are possible from any state other than "closed".

icestates-cont

@aboba aboba closed this as completed Jun 11, 2015
robin-raymond pushed a commit that referenced this issue Jun 22, 2015
Addressed Philipp Hancke's review comments, as noted in: Issue #198
894b08b 
Added the "failed" state to RTCIceTransportState, as noted in: Issue #199
Added text relating to handling of incoming media packets prior to remote fingerprint verification, as noted in: Issue #200
Added a complete attribute to the RTCIceCandidateComplete dictionary, as noted in: Issue #207
Updated the description of RTCIceGatherer.close() and the "closed" state, as noted in: Issue #208
Updated Statistics API error handling to reflect proposed changes to the WebRTC 1.0 API, as noted in: Issue #214
Updated Section 10 (RTCDtmfSender) to reflect changes in the WebRTC 1.0 API, as noted in: Issue #215
Clarified state transitions due to consent failure, as noted in: Issue #216
Added a reference to [FEC], as noted in: Issue #217
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
1.0 compatibility issue 1.1 PR exists
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@aboba @robin-raymond