ICE restart needed #59

Activity

1. martinthomson commented on Apr 14, 2014

   

   martinthomson

   on Apr 14, 2014

   Contributor

   More actions

   I'm not convinced that this is needed. I can see how it might be wanted, but not needed.

   But if it is included, tweaks on the remote side won't have the desired effect. Changes need to be local too. In a restart, as defined by 5245, the local ufrag and pwd change. To do that, maybe you should make a new ICEListener and push that in somehow.
2. robin-raymond

   added

   1.1

   on Apr 29, 2014
3. robin-raymond commented on Apr 29, 2014

   

   robin-raymond

   on Apr 29, 2014

   ContributorAuthor

   More actions

   Discussion was held here: http://lists.w3.org/Archives/Public/public-ortc/2014Apr/0127.html

   And decision (thus far) is that nothing is truly needed to add to API surface. Create "brand new" RTCIceListener/RTCIceTransport pair to do restart.

   We should have an example of ICE restart in specification somewhere.
4. robin-raymond

   added a commit that references this issue on May 14, 2014

   - ICE restart explanation added, as described in Issue 59

   ...

   30c585c
5. robin-raymond

   closed this as completedon May 14, 2014
6. juberti commented on May 15, 2014

   

   juberti

   on May 15, 2014

   More actions

   I don't think this is sufficient given the need for TransportController and freezing policy. The TransportController really needs to be able to redo its thing, suggesting that ICE restart needs to be internal to an IceTransport rather than configured externally. (Unless we want to go down the route of having multiple TCs, which starts getting into questionable territory).