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

"ICE Agent" and "ICE engine" versus "ICE Transport" #132

Closed
aboba opened this issue Jul 15, 2014 · 0 comments
Closed

"ICE Agent" and "ICE engine" versus "ICE Transport" #132

aboba opened this issue Jul 15, 2014 · 0 comments

Comments

@aboba
Copy link
Contributor

aboba commented Jul 15, 2014

In a number of places, the ORTC API uses terms such as "ICE Agent" and "ICE engine" that make sense in WebRTC 1.0, but do not make sense in ORTC API, where there are ICE Transport objects that can be configured individually (and can have separate states).

Proposed resolution:

In Section 3.9 change:

"all
The ICE engine gathers all types of candidates when this value is specified.

nohost
The ICE engine gathers all ICE candidate types except for host candidates.

relay
The ICE engine must only gather media relay candidates such as candidates passing through a TURN server. This can be used to reduce leakage of IP addresses in certain use cases."

To:

"
all
The ICE transport gathers all types of candidates when this value is specified.

nohost
The ICE transport gathers all ICE candidate types except for host candidates.

relay
The ICE transport must only gather media relay candidates such as candidates passing through a TURN server. This can be used to reduce leakage of IP addresses in certain use cases."

In Section 3.10, change:

"
new
The ICE Agent is gathering addresses and/or waiting for remote candidates to be supplied.

checking
The ICE Agent has received remote candidates on at least one component, and is checking candidate pairs but has not yet found a connection. In addition to checking, it may also still be gathering.

connected
The ICE Agent has found a usable connection for all components but is still checking other candidate pairs to see if there is a better connection. It may also still be gathering.

completed
The ICE Agent has finished gathering and checking and found a connection for all components. Open issue: it is not clear how the non controlling ICE side knows it is in the state.

disconnected
Liveness checks have failed for one or more components. This may trigger intermittently (and resolve itself without action).

closed
The ICE Agent has shut down and is no longer responding to STUN requests."

To:

"
new
The ICE Transport is gathering addresses and/or waiting for remote candidates to be supplied.

checking
The ICE Transport has received at least one remote candidate, and is checking candidate pairs but has not yet found a connection. In addition to checking, it may also still be gathering.

connected
The ICE Transport has found a usable connection, but is still checking other candidate pairs to see if there is a better connection. It may also still be gathering.

completed
The ICE Transport has finished gathering and checking and found a connection. There is no such state for the ICE non-controlling peer.

disconnected
Liveness checks have failed. This may trigger intermittently (and resolve itself without action).

closed
The ICE Transport has shut down and is no longer responding to STUN requests."

robin-raymond pushed a commit to robin-raymond/ortc that referenced this issue Jul 16, 2014
…3c#66

Added Identity support, as described in Issue w3c#78
Reworked getStats method, as described in Issue w3c#85
Removed ICE restart method described in Issue w3c#93
Addressed CNAME and synchronization context issues described in Issue w3c#94
Fixed WebIDL issues noted in Issue w3c#97
Addressed NITs described in Issue w3c#99
DTLS transport issues fixed as described in Issue w3c#100
ICE transport issues fixed as described in Issue w3c#101
ICE transport controller fixes made as described in Issue w3c#102
Sender and Receiver object fixes made as described in Issue w3c#103
Fixed RTCRtpEncodingParameter default issues described in Issue w3c#104
Fixed 'Big Picture' issues descibed in Issue w3c#105
Fixed RTCRtpParameter default issues described in Issue w3c#106
Added a multi-stream capability, as noted in Issue w3c#108
Removed quality scalability capabilities and parameters, as described in Issue w3c#109
Added scalability examples as requested in Issue w3c#110
Addressed WebRTC 1.0 Data Channel compatibility issue described in Issue w3c#111
Removed header extensions from RTCRtpCodecParameters as described in Issue w3c#113
Addressed RTP/RTCP non-mux issues with IdP as described in Issue w3c#114
Added getParameter methods to RTCRtpSender and RTCRtpReceiver objects, as described in Issue w3c#116
Added layering diagrams as requested in Issue w3c#117
Added a typedef for payload type, as described in Issue w3c#118
Moved onerror from the RTCIceTransport object to the RTCIceListener object as described in Issue w3c#121
Added explanation of Voice Activity Detection (VAD), responding to Issue w3c#129
Clarified the meaning of maxTemporalLayers and maxSpatialLayers, as noted in Issue w3c#130
Added RFC 6051 to the list of header extensions and removed RFC 5450, as noted in Issue w3c#131
Addressed ICE terminology issues, as described in Issue w3c#132
Separated references into Normative and Informative, as noted in Issue w3c#133
@aboba aboba closed this as completed Jul 17, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant