Turn RTP header extension ID mismatch into error

RESOLVED FIXED in Firefox 61

Status

()

P3
normal
Rank:
21
RESOLVED FIXED
a year ago
5 months ago

People

(Reporter: drno, Assigned: drno)

Tracking

Trunk
mozilla61
Points:
---

Firefox Tracking Flags

(firefox61 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

a year ago
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.
(Assignee)

Updated

a year ago
Rank: 21
Target Milestone: --- → mozilla57
Mass change P2->P3 to align with new Mozilla triage process.
Priority: P2 → P3
(Assignee)

Comment 2

7 months ago
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
(Assignee)

Comment 3

7 months ago
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)
(Assignee)

Comment 5

7 months ago
Turns out Cisco Spark is stripping the extmap from their SDP. So we can put the error back in.
(Assignee)

Updated

7 months ago
Attachment #8954659 - Flags: review?(docfaraday)

Comment 6

7 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+
(Assignee)

Comment 7

7 months ago
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

6 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

6 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/6bc7318ca790
Status: NEW → RESOLVED
Last Resolved: 6 months ago
status-firefox61: --- → fixed
Resolution: --- → FIXED
Target Milestone: mozilla57 → mozilla61
(Assignee)

Updated

5 months ago
Flags: needinfo?(drno)
You need to log in before you can comment on or make changes to this bug.