RESOLVED FIXED 184688
WebRTC DataChannel id of initiator not set 
https://bugs.webkit.org/show_bug.cgi?id=184688
Summary WebRTC DataChannel id of initiator not set
Danilo
Reported 2018-04-17 04:42:14 PDT
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.
Attachments
Demo (1.81 KB, application/gzip)
2018-04-17 04:42 PDT, Danilo 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Lennart Grahl
Comment 1 2018-07-26 09:57:44 PDT
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]
Comment 2 2022-06-21 14:36:57 PDT
Nowadays http://wpt.live/webrtc/RTCDataChannel-id.html
Radar WebKit Bug Importer
Comment 3 2022-06-21 14:37:10 PDT
<rdar://problem/95636476>
youenn fablet
Comment 4 2022-06-23 00:39:22 PDT
Pull request: https://github.com/WebKit/WebKit/pull/1718
EWS
Comment 5 2022-06-23 12:20:45 PDT
Committed 251795@main (4bb2fa75278d): <https://commits.webkit.org/251795@main> Reviewed commits have been landed. Closing PR #1718 and removing active labels.
Note You need to log in before you can comment on or make changes to this bug.