Open
Bug 1501456
Opened 7 years ago
Updated 3 years ago
ICE fails with Chrome/Safari with server reflexive candidates only
Categories
(Core :: WebRTC: Networking, defect, P3)
Tracking
()
NEW
People
(Reporter: drno, Unassigned)
References
Details
Attachments
(1 file)
|
209.29 KB,
application/octet-stream
|
Details |
I learned last week that ICE interop with Safari and Chrome is broken if the other side only provides server reflexive candidates (no TURN, no host candidates).
From looking at Firefox ICE logs (attached) it looks like Firefox is sending a STUN binding request. But when it receives the binding response message from Safari or Chrome the STUN transaction ID doesn't match the ID from the request. Firefox then keeps retransmitting it's binding requests, but every single STUN answer from Chrome/Safari come back with a new different transaction ID. So Firefox keeps ignoring the responses because they don't match pending STUN transaction.
I have no idea if or how this works Chrome <-> Chrome. Maybe they ignore STUN transaction IDs if the ufrag/pwd match?
| Reporter | ||
Updated•7 years ago
|
Priority: -- → P3
Comment 1•7 years ago
|
||
I'd love to see more debug from the chrome side. I think starting it with --enable-logging=1 --v=4 should give us enough.
The debug I am looking for should be easily grep-able by searching for "STUN ping":
https://cs.chromium.org/chromium/src/third_party/webrtc/p2p/base/port.cc?sq=package:chromium&dr=CSs&g=0&l=1270
This looks bad:
// If this is an authenticated STUN request, then signal unknown address and
// send back a proper binding response.
Also a pcap would be good for the trial :-)
Comment 2•7 years ago
|
||
We have to compare transaction IDs or else we'll get lots of messed-up stuff when there are multiple interfaces. Unless this is a recent regression, this seems hard to believe.
Regarding fippo's comment, that code is for handling pings from unknown remote candidates, and will create a new prflx remote candidate accordingly. Seems unrelated.
Updated•3 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•