RTCP support for RTSP stream output

Our RTSP server currently doesn't support RTCP, and therefore isn't RTSP 1.0 valid.

changed milestone to %0.8.5-final

RTCP is a MUST to detect client timeouts and stop streaming if the client died unexpectedly (see also #279 (closed)).

This application has RTCP support. It could probably be integrated relativly easily. http://streaming.polito.it/server

changed milestone to %Features paradize

Some RTCP code was added recently so might as well bring this ticket to 0.8.6.

Original author: tumu

added Difficulty::medium label

changed milestone to %0.8.6

More clear summary.

Original author: tumu

changed title from RTCP for the vlm RTSP server to RTCP support

changed milestone to %Features paradize

Valid RTSP/1.0 ? yeah right, even the IETF admits that RTSP version 1.0 is a broken protocol (read the RTSP/1.1 summary).

We could try to be compatible, and use RTCP for RTSP unicast streaming keep-alive... But would this not break some clients? OTOH, the current RTCP-less stuff is broken anyhow.

assigned to @Courmisch

changed milestone to %0.9.0 features freeze

unassigned @Courmisch

In fact, RTCP is optional on the client side: the client can use RTSP and/or RTCP to refresh the timeout. However, on the RTSP server side it's mandatory, since some clients might be using to notify the server they are still up.

Currently, we probably do not really need RTCP in the RTP access, which is biased toward multicast or manually setup UDP streams anyway. And we do send RTCP from the RTP access output - we don't really care what we receive in that case for similar reasons.

However, most of the work would be in the RTSP server where there is currently nothing, meaning RTSP timeouts are currently broken.

changed title from RTCP support to RTCP support for RTSP stream output

changed milestone to %Features paradize

assigned to @Courmisch

changed milestone to %0.9.0 features freeze

closed

Ok, now we do have RTCP-SR, and we do tell clients not to send RTCP-RR, so I consider this issue closed.

added Status::fixed label

mentioned in issue #279 (closed)

mentioned in issue #568 (closed)