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)

63 Branch
defect

Tracking

()

People

(Reporter: drno, Unassigned)

References

Details

Attachments

(1 file)

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?
Priority: -- → P3
See Also: → 1496687
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 :-)
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.
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: