[vlc-devel] [PATCH 2/2] live555: Use OPTIONS as keep-alive fallback

[Paul Clark]

On 18/06/15 16:33, Julian Scheel wrote:
> So far no keep-alive messages are sent when GET_PARAMETERS is not supported by
> the server. Change this to fall back to sending OPTIONS as keep-alive message,
> which is the default keep-alive message in the SAT>IP specification. As
> RFC2326 does not enforce a specific command as keep-alive message this shall
> be valid for any compliant RTSP server.

I've seen clients (notably Samsung SmartTVs) do this against our server, 
so we support it as well as GET_PARAMETER, but it requires the client to 
send the Session: header with OPTIONS (which is pretty weird RTSP, 
really, but works), and I don't think your patch can be doing so:

+            p_sys->rtsp->sendOptionsCommand(NULL, NULL);

Does the SAT>IP spec specify this?  If not then they are mis-using a 
connection-level keepalive to trigger a session-level one, which is 
*really* bad RTSP...

Kind regards

Paul