[FFmpeg-devel] [PATCH 2/2] avformat/http: exponentially delay reconnection to avoid flooding the server

Michael Niedermayer michaelni at gmx.at
Mon Sep 7 19:41:19 CEST 2015


On Mon, Sep 07, 2015 at 10:53:15AM +0800, Zhang Rui wrote:
> >> Doesn't this have overflow issues?
> >> More precisely, are you sure this behaves correctly when the int
> >> reconnect_delay wraps around?
> >
> > i intend to add a reconnect_max that limits reconnects and prevents
> > overflow, just wanted to first make sure noone has objections to
> > these 2 patches before i write more code on top
> 
> These 2 patches are OK for me.

applied


> 
> BTW:
> Is it OK to expose generic io_callback in URLContext
> to API users to handle such staffs?
> Something like open_cb in AVFormatContext.
> 
> API users may need various check/action at reconnection,
> e.g. checking cellular/wifi, renewing expired url,
> injecting custom IO into nested protocols, alert end user.

sounds like it would make sense but i dont know what the other devels
oppinion about that is ...

thanks

[...]
--
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are too smart to engage in politics are punished by being
governed by those who are dumber. -- Plato 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150907/313a242d/attachment.sig>


More information about the ffmpeg-devel mailing list