Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

What should defaultIceServers be if browser has no default servers? #626

Closed
a2sheppy opened this issue May 6, 2016 · 5 comments
Closed

Comments

@a2sheppy
Copy link

a2sheppy commented May 6, 2016

The RTCPeerConnection.defaultIceServers attribute returns a list of ICE servers configured into the browser for use by default. However, the spec doesn't say what the browser should return if it doesn't offer default servers. Should this be null or an empty list?

@jan-ivar
Copy link
Member

jan-ivar commented May 7, 2016

The attribute is not nullable so it cannot be null, therefore it must return an empty list.

@a2sheppy
Copy link
Author

a2sheppy commented May 7, 2016

Fah, of course. I totally missed it reading the WebIDL and was looking
for it in the explanatory text. Thanks for pointing it out.

The attribute is not nullable so it cannot be null, therefore it must
return an empty list.


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#626 (comment)

Sheppy

@adam-be
Copy link
Member

adam-be commented May 11, 2016

Let's close this issue. @a2sheppy, please reopen if you disagree.

@a2sheppy
Copy link
Author

I'm fine with closing it, although I do feel that adding a couple of
words to the text of the spec to more explicitly call this out would be
helpful. But it probably doesn't matter to most of the people who will
actually read the spec. Those of us referring to it while writing
documentation about coding with WebRTC. :)

Let's close this issue. @a2sheppy https://github.com/a2sheppy,
please reopen if you disagree.


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#626 (comment)

Sheppy

@alvestrand
Copy link
Contributor

Closing as "it is clear".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants