Closed Bug 786313 Opened 12 years ago Closed 9 years ago

Automatic prioritization of interfaces for nICEr

Categories

(Core :: WebRTC: Networking, defect)

defect
Not set
normal

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.
Fixed in: : http://hg.mozilla.org/projects/alder/rev/b1a6785f8e0c

r=me
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: [qa-]
The code in nricectx.cpp probably still needs some cleanup
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Whiteboard: [qa-] → [WebRTC], [blocking-webrtc-]
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-]
I'm not exactly sure what cleanup we're talking about here. I can try to figure out.
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).
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.
Ok, let's go ahead and close this since we have the follow-up bugs.
Status: REOPENED → RESOLVED
Closed: 12 years ago9 years ago
Flags: needinfo?(docfaraday)
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.