Closed Bug 1120365 Opened 9 years ago Closed 9 years ago

"media-fail" indicates the wrong thing on desktop/standalone for direct calls

Categories

(Hello (Loop) :: Client, defect, P4)

Product:
Hello (Loop)
Component:
Client
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX
Blocking Flags:
backlog backlog+

People

(Reporter: standard8, Unassigned)

References

Details

(Whiteboard: [errors])

User Story

Notes for devs:

- gUM can only really fail for incoming calls; for outgoing/standalone we shouldn't be activating the call until after having got gUM.

- ICE failures. Currently the only way to get this seems to be to listen to the sdk OT.exception for "Unable to publish" code 1500 (one of the error codes listed in the sdk).
On Desktop, we're currently only sending media-fail as a termination reason on the websocket when the call is hungup, whilst media connection is still taking place.

We should be sending it when:

- gUM fails
- ICE fails

I'm not sure how we detect ICE at the moment (via the sdk), so we may need to split that to a separate bug.
We should address this soon as it affects metrics
Severity: enhancement → normal
backlog: --- → Fx38+
Priority: -- → P2
IMO we should prioritize this early in Fx38 and strongly consider uplifting (depending on the risk of the patch).

Shell -- Can you put this on the trello board?
Flags: needinfo?(sescalante)
User Story: (updated)
added to trello
Flags: needinfo?(sescalante)
Priority: P2 → P1
metrics are important - but rooms use is going up - so rooms use metrics are higher priority than direct calling.
Priority: P1 → P4
(In reply to sescalante from comment #4)
> metrics are important - but rooms use is going up - so rooms use metrics are
> higher priority than direct calling.

Whilst that's true, I think there's information in direct calls that is going to feed into if rooms are having problems or not - we don't get the same visibility with rooms.

I'd suggest this should be a P2 or P3.
As part of fixing this bug, the media-fail doc at <https://docs.services.mozilla.com/loop/apis.html#termination-reasons> should be updated.
backlog: Fx38+ → backlog+
Rank: 48
Flags: firefox-backlog+
Whiteboard: [errors]
We are currently reworking the Loop "user journey" (bug 1209713) and as part of this direct calls and contacts are being removed. Therefore closing direct call related bugs as wontfix.

Tracking id: directcallclosing
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.