Turn RTP header extension ID mismatch into error

RESOLVED FIXED in Firefox 61

Status

()

P3
normal
Rank:
21
RESOLVED FIXED
2 years ago
8 months ago

People

(Reporter: drno, Assigned: drno)

Tracking

Trunk
mozilla61
Points:
---

Firefox Tracking Flags

(firefox61 fixed)

Details

Attachments

(1 attachment)

In bug 1355010 we temporarily reduced the error when the answerer changes RTP header extension IDs into a warning.

Once Cisco Spark no longer changes ID's in it's answerer we should turn it back into an error.
Rank: 21
Target Milestone: --- → mozilla57
Mass change P2->P3 to align with new Mozilla triage process.
Priority: P2 → P3
The bug number in the description is wrong: It was not bug 1355010 but bug 1361206 where we turned of the RTP header extension ID matching check
Looks like we can turn this back into an error... because Cisco Spark no longer negotiates any header extensions at all?!
Comment hidden (mozreview-request)
Turns out Cisco Spark is stripping the extmap from their SDP. So we can put the error back in.
Attachment #8954659 - Flags: review?(docfaraday)

Comment 6

10 months ago
mozreview-review
Comment on attachment 8954659 [details]
Bug 1363900: re-enable extmap ID remapping error.

https://reviewboard.mozilla.org/r/223758/#review230236
Attachment #8954659 - Flags: review?(docfaraday) → review+
I guess this will have to wait until the soft freezes is lifted and 61 is going...
Assignee: nobody → drno
Flags: needinfo?(drno)
Comment hidden (mozreview-request)

Comment 9

9 months ago
Pushed by drno@ohlmeier.org:
https://hg.mozilla.org/integration/autoland/rev/6bc7318ca790
re-enable extmap ID remapping error. r=bwc

Comment 10

9 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/6bc7318ca790
Status: NEW → RESOLVED
Last Resolved: 9 months ago
status-firefox61: --- → fixed
Resolution: --- → FIXED
Target Milestone: mozilla57 → mozilla61
Flags: needinfo?(drno)
You need to log in before you can comment on or make changes to this bug.