search

nickdesaulniers / node-rtc-peer-connection

[BROKEN/UNMAINTAINED] RTCPeerConnection for Node.js
Other
154 stars 17 forks source link

eliminate redundate candidates #39

Closed nickdesaulniers closed 8 years ago

nickdesaulniers commented 8 years ago

https://tools.ietf.org/html/draft-ietf-ice-rfc5245bis-00#section-4.1.3

during the gathering phase, it's possible that host candidates match srflx candidates, so we should eliminate them in that case.

nickdesaulniers commented 8 years ago

debug.printValidPairs shows:

...
2620:101:80fc:232:2acf:e9ff:fe1f:3939:59505 -> 2620:101:80fc:232:2acf:e9ff:fe1f:3939:64741
2620:101:80fc:232:2acf:e9ff:fe1f:3939:59505 -> 2620:101:80fc:232:2acf:e9ff:fe1f:3939:64741
...

which are duplicates, and

...
63.245.221.34:55012 -> 10.252.35.64:53671
...

which is a srflx.