[Live-devel] [PATCH] Declare RTSP server's timeout

[Warren Young]

Ross Finlayson wrote:
> Our server impelmentation uses both RTSP commands *and* RTCP ("RR") 
> packets from clients to indicate liveness.  Therefore, the "timeout" 
> parameter in a "Session:" header is not needed, because periodic RTSP 
> commands (e.g., "GET_PARAMETER") from the client are not needed in order 
> to tell the server that the session is alive.  Instead, the server gets 
> this information from RTCP reports.

I have a client here that doesn't send RTCP RR packets to the server. 
It does send GET_PARAMETER, every ~60 seconds unless told different. 
Therefore, the server times out, since its hard-coded timeout is 
shorter.  If the server tells the client that it has a shorter timeout 
than the RFC recommends as a default, it would send its GET_PARAMETER 
commands more often, so the connection wouldn't time out.

I would think this patch would be a no-brainer.  The spec says "do X," 
and the patch makes the server do X.  What's the downside?