884196 - ICE: report error on network interface change

Status: NEW

(Core :: WebRTC: Networking, defect, P4)

Description

[Shian-Yow Wu [:swu]]

Prior to bug 881982, we should at least report error when network interface changed.

Comment 1

[u459114]

Hi Shian-Yow,
Is there a more detailed plan on error report?
1. Error report mechanism
2. Error handling mechanism
3. Category of error kind, and a high level priotized list.

Comment 2

[Shian-Yow Wu [:swu]]

(In reply to C.J. Ku[:CJKu] from comment #1)
> Hi Shian-Yow,
> Is there a more detailed plan on error report?
> 1. Error report mechanism

When nICEr detects the error, the report path will be:
  nICEr -> mtransport -> PeerConnection -> App

According to Ekr, 
nICEr already has mechanisms for this, namely:
- call the stream_failed() handler on the stream [ice_handler.h]
And ICE_CTX_FAILED will be done in the handler in NrIceCtx.

> 2. Error handling mechanism
The events could be just a notification, not necessarily to be error.
For example, if ICE module received interface change event, it may decide to get network interface list again.

> 3. Category of error kind, and a high level priotized list.
Example of events:
- Interface up/down
- IP address change (and/or routing/DNS change?)
- ICMP error
- Lost of connectivity

Comment 3

[Bulk Bug Changes for mreavy's org]

Mass change P3->P4 to align with new Mozilla triage process.