Closed
Bug 786313
Opened 12 years ago
Closed 9 years ago
Automatic prioritization of interfaces for nICEr
Categories
(Core :: WebRTC: Networking, defect)
Core
WebRTC: Networking
Tracking
()
RESOLVED
FIXED
backlog | parking-lot |
People
(Reporter: ekr, Unassigned)
References
Details
Currently nICEr has fixed prioritization for each interface. This chokes if unknown interface names are provided. At minimum we need a way to automatically prioritize new interfaces. Better would be to interrogate the interface and try to automatically prioritize everything.
Reporter | ||
Comment 1•12 years ago
|
||
Fixed in: : http://hg.mozilla.org/projects/alder/rev/b1a6785f8e0c r=me
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
Whiteboard: [qa-]
Reporter | ||
Comment 3•12 years ago
|
||
The code in nricectx.cpp probably still needs some cleanup
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Updated•12 years ago
|
Whiteboard: [qa-] → [WebRTC], [blocking-webrtc-]
Comment 4•9 years ago
|
||
Byron - if there's more cleanup needed, please file a bug for it and set the priority/rank/etc; we should likely close this bug.
backlog: --- → parking-lot
Flags: needinfo?(docfaraday)
Whiteboard: [WebRTC], [blocking-webrtc-]
Comment 5•9 years ago
|
||
I'm not exactly sure what cleanup we're talking about here. I can try to figure out.
Reporter | ||
Comment 6•9 years ago
|
||
This is done, though it could be improved. Right now we use a sort of ad hoc order on Mac and Windows (some combination of defined names and auto-assignment). We should extend nrinterfaceprioritizer to these platforms (which means exposing interface types on them).
Reporter | ||
Comment 7•9 years ago
|
||
Looking at this code, the net net is that we have semi-sensible defaults for the common interface names on Mac and probably just do order of encountering on windows (because the defaults are only for mac names). The thing we're missing to make the prioritizer work on those platforms is exporting the interface properties that it uses.
Reporter | ||
Comment 8•9 years ago
|
||
See also: https://bugzilla.mozilla.org/show_bug.cgi?id=895790 and: https://bugzilla.mozilla.org/show_bug.cgi?id=895793
Comment 9•9 years ago
|
||
Ok, let's go ahead and close this since we have the follow-up bugs.
Status: REOPENED → RESOLVED
Closed: 12 years ago → 9 years ago
Flags: needinfo?(docfaraday)
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•