W3C home > Mailing lists > Public > public-webrtc-logs@w3.org > July 2019

Re: [webrtc-pc] restartIce(): Look at only ufrag or (ufrag,pwd)? (#2229)

From: Jan-Ivar Bruaroey via GitHub <sysbot+gh@w3.org>
Date: Wed, 17 Jul 2019 01:29:34 +0000
To: public-webrtc-logs@w3.org
Message-ID: <issue_comment.created-512064466-1563326973-sysbot+gh@w3.org>
[section 5.9](https://tools.ietf.org/html/draft-ietf-rtcweb-jsep-26#section-5.9) says: *"Or, if the ICE ufrag and password values have changed, trigger the ICE agent to start an ICE restart"*

English is imprecise, but I think had the intent been to not trigger unless *both* values have changed, then it would have said so: E.g. *"if the ICE ufrag and password values have both changed"*

This is also SLD, where it would be hard to not have both values change when they change.

I'd put more stock in 5.10 (SRD) where it needs to consider invalid inputs. Thus I'd interpret the intent here to be to trigger if the values *as a group* have changed, meaning *either or both* of the values have changed.

It also makes more sense to me that way. If we don't restart when ice-pwd changes, we would need to figure out what should happen then.

Adding a PR.

-- 
GitHub Notification of comment by jan-ivar
Please view or discuss this issue at https://github.com/w3c/webrtc-pc/issues/2229#issuecomment-512064466 using your GitHub account
Received on Wednesday, 17 July 2019 01:29:36 UTC

This archive was generated by hypermail 2.4.0 : Friday, 17 January 2020 19:22:25 UTC