WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
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.
Top of Page
Format For Printing
XML
Clone This Bug