Open Bug 1257006 Opened 8 years ago Updated 2 years ago

iceServers should get pruned for dupes

Categories

(Core :: WebRTC, defect, P3)

Product:
Core
Component:
WebRTC
Type:
defect

Tracking

()

Blocking Flags:
backlog webrtc/webaudio+

People

(Reporter: drno, Unassigned)

References

Details

I need to double check this, but I think we currently happily accept dupes of the same STUN/TURN server in |iceServers| and will then try to connect to all copies from all of applicable interfaces.
We should scan |iceServers| for dupes and remove/ignore them.

If we get passed in an IP address and a DNS name pointing to the same IP I'm cool letting that pass. The effort for catching that is probably too high.

The downside of filtering out dupes could be that someone might put in the same DNS name multiple times, assuming that multiple DNS resolutions results in different IP addresses (assuming the DNS name points to multiple IPs). But if someone tries that on purpose I think he/she should find a different solution for providing multiple STUN/TURN servers.
backlog: --- → webrtc/webaudio+
Rank: 29
Mass change P2->P3 to align with new Mozilla triage process.
Priority: P2 → P3
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.