- 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
@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