Closed Bug 1339203 Opened 6 years ago Closed 11 months ago

RTCPeerConnection does not support setting rtcpMuxPolicy parameter

Categories

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

51 Branch
defect

Tracking

()

RESOLVED DUPLICATE of bug 1342206

People

(Reporter: juandebravo, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [spec-compliance])

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/57.0.2987.37 Safari/537.36

Steps to reproduce:

Create an RTCPeerConnection object configuring the rtcpMuxPolicy value:

// rtcp-mux is optional
new RTCPeerConnection({rtcpMuxPolicy: "negotiate"})

or

// rtcp-mux is mandatory
new RTCPeerConnection({rtcpMuxPolicy: "require"})


Actual results:

RTCP candidates are always gathered, no matter the rtcpMuxPolicy value


Expected results:

When rtcpMuxPolicy is "require", RTCP candidates should not be gathered, as rtcp-mux is mandated.

Based on spec: https://w3c.github.io/webrtc-pc/#rtcrtcpmuxpolicy-enum
Component: Untriaged → WebRTC: Networking
Product: Firefox → Core
Status: UNCONFIRMED → NEW
backlog: --- → webrtc/webaudio+
Rank: 25
Ever confirmed: true
Priority: -- → P2
Whiteboard: [spec-compliance]
Mass change P2->P3 to align with new Mozilla triage process.
Priority: P2 → P3
Depends on: 1402221
Blocks: 1533017

The spec here changed two years ago, and "negotiate" was dropped due to "lack of implementer interest". Even though it's what Firefox implements, this wasn't caught at TPAC 2019 because we didn't have the prerequisite WebIDL in place, we simply just implemented this by default, in spite of the spec at the time requiring that "require" be the default value. We also didn't implement any of the transport objects, and this would have required and extra one: rtcpTransport. Other browsers never supported this.

"require" remains the default, but is now also the sole option.

Status: NEW → RESOLVED
Closed: 11 months ago
Resolution: --- → DUPLICATE
Duplicate of bug: 1342206
You need to log in before you can comment on or make changes to this bug.