Re: [MMUSIC] ICE-SIP-SDP: Concluding ICE statement

[Christer Holmberg <[e-mail]>]

Hi,

>>>>>This makes sense, but I am not sure it is precise enough. What we are trying to say is that ICE nomination
>>>>>process is done (completed or failed) for all streams and no new candidate pairs are going to be nominated.
>>>> 
>>>> The text talks about when the nomination starts.
>>>> 
>>>>>Starts or ends?
>>>>> 
>>>>>The sentence in 3.3.4 was supposed to mean that when ICE nomination process completes and if for any stream non-default 
>>>>>>candidate was nominated and ice2 option was not set, to send a new offer.
>>>> 
>>>> It also says that :)
>>>> 
>>>> But, the first sentence says:
>>>> 
>>>>“Once the state of each check list is Completed, and if the agent is
>>>>the controlling agent, it nominates a candidate pair [RFC8445]…”
>>>> 
>>>>…which at least to me describes when the nomination starts.
>>> 
>>> This is actually still about the nomination end. Essentially, what is described here is at the end of the nomination process, 
>>> for each stream a candidate pair is nominated (or the stream failed). If ice2 option is not set, and pair nominated in the end 
>>> is not the default candidate pair for this stream, then new offer should be sent to "true up" default candidates.
>>> 
>>>What this whole language is missing is that for some streams nomination can fail without failure of the entire session. Stream failure 
>>>should also result in a new offer with the failed streams disabled (port set to 0). I am not sure this is specified anywhere. Another thing 
>>>which is not specified anywhere if new offer due to stream failure is only required if ice2 option is not set or always. I would vote for 
>>>always to make sure this m= line can be reused.
>> 
>> Not sure what you mean by “nominating process”. My understanding of “nomination” is when the agent picks a 
>> Valid Pair and sends a STUN request with the nomination bit set.
>> 
>> But, in any case the “Once the state of each check list is Completed,” statement is wrong.  Of course an implementation might mandate 
>> that each check list must be Completed in order to establish the session, but there is no requirement in the ICE standard that each check 
>> list has to be Completed in order to do so
>
> The language is about end of ICE process, when all candidate lists are in either Completed or Failed state.
>
> Nominated pair at the time the list is in the Completed state is the final pair used for communications.

Not sure what you mean by "final pair". There might be many valid pairs associated with a check list, and the agent picks which it wants to nominate. That then becomes the "valid pair".

Maybe we could say:

   "Once the state of *A* check list is Completed, and if the agent is
   the controlling agent, it nominates a candidate pair [RFC8445] and
   checks whether the nominated pair matches the
   default candidate pair associated with the check list."

Regards,

Christer