Re: [webrtc-extensions] What does it mean to NOT nominate candidatePair? (#179)

"not nominate" means that the ICE agent should continue to perform connectivity checks and maybe propose another candidate pair for nomination.

[RFC8445 section 8.1.1](https://www.rfc-editor.org/rfc/rfc8445#section-8.1.1) permits this flexibility:

> The criteria for stopping the connectivity checks and for picking a pair for nomination are outside the scope of this specification. They are a matter of local optimization.

So whether a cancelled nomination is a ban or a deferral for a certain period is an implementation detail of the ICE agent. This shouldn't affect interoperability because the exact behaviour wasn't constrained to begin with by RFC8445 (or by webrtc-pc).

The non-normative note could be extended to clarify that the same candidate pair could be nominated again either by the ICE agent or as a result of setSelectedCandidatePair.


> It continues: _"The ICE agent may continue to send data using the candidate pair indicated by candidatePair unless instructed to use another candidate pair with setSelectedCandidatePair."_
> 
> Is this _"may"_ a normative MAY, or a MUST? If not, we could use _"will"_ and use a non-normative note if this is just an observation.
> 
> It's a bit tricky since we seem to be modifying ICE agent behavior here.

This is not a normative statement, and the intention is not to modify ICE agent behaviour. This refers to [RFC8445 section 12.1](https://www.rfc-editor.org/rfc/rfc8445#section-12.1):

> An ICE agent MAY send data on any valid pair before selected pairs have been produced for the data stream.

So your suggestion to change to "will" and extract out the observation into a non-normative note makes sense.

I can draft a PR with the clarifications.

-- 
GitHub Notification of comment by sam-vi
Please view or discuss this issue at https://github.com/w3c/webrtc-extensions/issues/179#issuecomment-1787362491 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Tuesday, 31 October 2023 14:44:04 UTC