Can rtpReceiver.rtcpTransport retrieve null? #349

Closed
ibc opened this Issue Jan 14, 2016 · 4 comments

Projects

None yet

2 participants

@ibc
Contributor
ibc commented Jan 14, 2016
var transport = new RTCDtlsTransport(...);
var rtpReceiver = new RTCRtpReceiver(transport);

rtpReceiver.transport;
// => transport

rtpReceiver.rtcpTransport;
// => transport or null or undefined?
@aboba aboba added 1.1 1.0 to-do-next-draft and removed 1.0 labels Jan 14, 2016
@aboba
Contributor
aboba commented Jan 18, 2016

I believe that rtcpTransport should be unset in this case since an rtcpTransport was not supplied in the constructor.

@ibc
Contributor
ibc commented Jan 18, 2016

Another approach would be for the getter to retrieve the effective RTCDtlsTransport, which at the end is the given transport.

IMHO it should be clarified in the spec.

@aboba
Contributor
aboba commented Jan 19, 2016

Just found the corresponding text in WebRTC 1.0.

Section 5.2.1 (RTCRtpSender)

The rtcpTransport attribute is the transport over which RTCP is sent and received. When BUNDLE is used, many RTCRtpSender objects will share one rtcpTransport and will all send and receive RTCP over the same transport. When RTCP mux is used, rtcpTransport will be null, and both RTP and RTCP traffic will flow over the transport described by transport.

Section 5.3.1 (RTCRtpReceiver)

The RTCRtpReceiver.rtcpTransport attribute is the transport over which RTCP is sent and received. When BUNDLE is used, many RTCRtpReceiver objects will share one RTCRtpReceiver.rtcpTransport and will all send and receive RTCP over the same transport. When RTCP mux is used, RTCRtpReceiver.rtcpTransport will be null, and both RTP and RTCP traffic will flow over RTCRtpReceiver.transport.

@ibc
Contributor
ibc commented Jan 19, 2016

Clear. Thanks.

@aboba aboba added a commit that referenced this issue Jan 19, 2016
@aboba aboba Can rtpReceiver.rtcpTransport retrieve null?
Sync definitions of RTCRtpReceiver.rtcpTransport and RTCRtpReceiver.rtcpTransport with WebRTC 1.0. 

Fix for Issue #349
ae5b0f9
@aboba aboba added PR exists and removed to-do-next-draft labels Jan 19, 2016
@aboba aboba closed this Jan 20, 2016
@aboba aboba reopened this Jan 20, 2016
@aboba aboba closed this Jan 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment