[libav-devel] [FFmpeg-devel] [PATCH] handle RTSP server keep-alive

Luca Barbato lu_zero at gentoo.org
Mon Feb 27 11:03:09 CET 2012


On 2/27/12 9:26 AM, Martin Storsjö wrote:
> On Mon, 27 Feb 2012, Luca Barbato wrote:
>
>> On 2/26/12 10:54 PM, Martin Storsjö wrote:
>>> On Sat, 25 Feb 2012, Tommy Winther wrote:
>>>
>>>> Yes, this fixes playback for me.
>>>> I think your patch is more complete than mine,
>>>> so it would be nice to get that added.
>>>
>>> Great, thanks for testing.
>>>
>>> Luca, do you think the patch attached to the mail earlier in this thread
>>> is ok?
>>>
>>
>> It should mention it adds support for OPTIONS requests.
>
> Actually, my original version of this patch sent "501 not implemented"
> as reply to all requests - that was enough to "Handle requests from
> server to client" as in not fuck up the state and be confused about what
> has happened, as it did before :-) The case I wrote the code for
> initially worked just fine with such a stub reply, but the patch by
> Tommy replied with 200 ok, so I changed it to do such replies for
> options for now.
>
> Anyway, ok to push with the message amended?
>

Sure, next should be factored a bit since that function is growing too 
much. Please push with the amended message for now =)



More information about the libav-devel mailing list