W3C home > Mailing lists > Public > public-webrtc-logs@w3.org > August 2018

Re: [webrtc-pc] Is an RTCIceServer with no urls useful?

From: Jan-Ivar Bruaroey via GitHub <sysbot+gh@w3.org>
Date: Thu, 30 Aug 2018 13:51:16 +0000
To: public-webrtc-logs@w3.org
Message-ID: <issue_comment.created-417326740-1535637075-sysbot+gh@w3.org>
@steveanton Given that `urls` is a required field that can be either a string or an array, we have
```js
{ iceServers: [ { urls: "" } ] } // SyntaxError
{ iceServers: [ { urls: [] } ] } // success
```

Perhaps we should throw `SyntaxError` for both?

As it stands now, our [parsing steps](http://w3c.github.io/webrtc-pc/#set-pc-configuration) appear to do the wrong thing with the latter:

   *11. Append server to validatedServers.*

-- 
GitHub Notification of comment by jan-ivar
Please view or discuss this issue at https://github.com/w3c/webrtc-pc/issues/1949#issuecomment-417326740 using your GitHub account
Received on Thursday, 30 August 2018 13:51:17 UTC

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