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

RTCPeerConnectionIceErrorEvent errorCode not defined #521

Closed
stefhak opened this issue Feb 25, 2016 · 2 comments
Closed

RTCPeerConnectionIceErrorEvent errorCode not defined #521

stefhak opened this issue Feb 25, 2016 · 2 comments
Assignees

Comments

@stefhak
Copy link
Contributor

stefhak commented Feb 25, 2016

https://www.w3.org/2011/04/webrtc/wiki/images/5/50/WebRTCWG-2016-02-25.pdf#9 lists the error codes, but the spec does not provide that info (just says the error is a string).

@aboba aboba self-assigned this Feb 26, 2016
aboba added a commit that referenced this issue Feb 26, 2016
@robin-raymond
Copy link

4.9.3.2 Attributes
If the server could not be reached, errorCode will be set to a TBD value in the 7XX range, as this does not conflict with the STUN error code range.

I'm not sure this particular error is wise. In any implementation (especial IPV6), it's very likely many host candidates will not be able to reach the TURN server or STUN server because they don't route appropriately. So you will warn/flag these as being failed but with other host candidates they actually succeed. The programmer will think things are going horribly wrong when this is probably normal.

Further, if you don't indicate that this URI is actually succeeding then the programmer is going to be somewhat in the dark as to what's going on.

I would only use the "cannot contact server" error if NONE of the host IPs could reach the server. (Although that could just indicate a general connectivity problem and not really a server problem).

aboba added a commit that referenced this issue Mar 10, 2016
Removed "server cannot be reached" error, responding to Robin Raymond's comment on Issue #521
@alvestrand
Copy link
Contributor

Fixed in #525

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

4 participants