search

w3c / webrtc-ice

Extension to the RTCIceTransport interface defined in WebRTC 1.0
https://w3c.github.io/webrtc-ice/
Other
9 stars 14 forks source link

addRemoteCandidate: Action on illegal candidate not specified #19

Closed alvestrand closed 4 years ago

alvestrand commented 6 years ago

The specification of addRemoteCandidate() doesn't say what it should do if the candidate is malformed or otherwise illegal.

The webrtc-pc spec says that addIceCandidate() should return operationError in this case; "parameter error" would seem more logical, but consistency is probably more important than logic.

aboba commented 6 years ago

@alvestrand Agree that consistency is more important. Also, for consistency, addRemoteCandidate should take RTCIceCandidateInit as an argument, rather than RTCIceCandidate, similar to addIceCandidate in WebRTC 1.0.