Bug 184688 – WebRTC DataChannel id of initiator not set

• Component: WebRTC
• Status: RESOLVED FIXED

[Danilo]

Created attachment 338098 [details]
Demo

When creating a WebRTC DataChannel with in-band announcement (negotiated=false), the "id" attribute is not set on the side of the initiator.

Demo: https://tmp.dbrgn.ch/dc/

If you open this in Chromium or Firefox, the data channel id will be reported as an integer on both sides.

> DC1 is open. id=1
> DC2 is open. id=3
> New data channel. id=1
> New data channel. id=3

If you open it in Safari 11 or in the current technology preview, it will be reported as null on the initiator side.

> DC1 is open. id=null
> DC2 is open. id=null
> New data channel. id=1
> New data channel. id=3

The JS sourcecode can be found at https://tmp.dbrgn.ch/dc/dc.js.

From the spec (https://www.w3.org/TR/webrtc/#dom-rtcdatachannel-id):

> The id attribute returns the ID for this RTCDataChannel . The value is initally null, which is what will be returned if the ID was not provided at channel creation time, and the DTLS role of the SCTP transport has not yet been negotiated. Otherwise, it will return the ID that was either selected by the script or generated by the user agent according to [ RTCWEB-DATA-PROTOCOL].

The current behavior seems to be a bug, and it breaks one of our data channel based applications in Safari.

[Lennart Grahl]

My upcoming WPT tests confirm this bug: https://github.com/web-platform-tests/wpt/pull/10468

FWIW, if I hadn't worked around this bug, pretty much all of the data channel tests would fail. So, I would appreciate if you fix it, so I can remove the workaround from the WPT test suite.

[Sam Sneddon [:gsnedders]]

Nowadays http://wpt.live/webrtc/RTCDataChannel-id.html

[Radar WebKit Bug Importer]

<rdar://problem/95636476>

[youenn fablet]

Pull request: https://github.com/WebKit/WebKit/pull/1718

[EWS]

Committed 251795@main (4bb2fa75278d): <https://commits.webkit.org/251795@main>

Reviewed commits have been landed. Closing PR #1718 and removing active labels.