1529588 - ICE server validation does not catch most problems

Status: NEW

(Core :: WebRTC: Signaling, defect, P2)

Description

[Byron Campen [:bwc]]

This is keeping us from enabling a number of web-platform-tests.

https://jsfiddle.net/6x90jvwp/

Comment 1

[Nico Grunbaum [:ng, @chew:mozilla.org]]

https://github.com/w3c/webrtc-pc/issues/3049

Comment 2

[Byron Campen [:bwc]]

Attachment: Bug 1529588: Bring ICE server validation up-to-spec. r?jib,mjf

Comment 3

[Byron Campen [:bwc]]

Attachment: Bug 1529588: Remove invalid ICE server from this list. r?jmaher

Comment 4

[Byron Campen [:bwc]]

Attachment: Bug 1529588: Replace NrIceStun/TurnServer with common type. r?mjf

There is no need for this intermediate representation anymore.

Claude wrote the bulk of this, with guidance and edits from me.

Comment 5

[Byron Campen [:bwc]]

Attachment: Bug 1529588: Stop allowing empty credential, and update tests to reflect latest spec. r?jib