[FFmpeg-devel] Re: [PATCH] tcp: add TCP keepalive tuning options

[Nicolas George via ffmpeg-devel <ffmpeg-devel@ffmpeg.org>]

Practice2001 via ffmpeg-devel (HE12025-11-21):
> Addition of tcp_keepalive, tcp_keepidle, tcp_keepintvl, and tcp_keepcnt
> support to the TCP protocol. Exposeing these options to the HTTP protocol
> so they can be used for HTTP(S) connections. Updated documentation.
> Tested with: ./configure && make && make fate

I am rather against this. We already have the option to enable it, which
is necessary.

But tweaking the parameters… Every time I have seen somebody want to do
it, it was for the wrong reasons, to control user-visible timeouts.
These settings are not designed for that and do not offer enough control
to do it accurately.

The fact that the proposal talks of using it in HTTP but not how is not
encouraging.

The fact that it does not work on Windows is also an indication: we
cannot rely on them to implement features.

Let us just close #11671 as wontfix.

Regards,

-- 
  Nicolas George
_______________________________________________
ffmpeg-devel mailing list -- ffmpeg-devel@ffmpeg.org
To unsubscribe send an email to ffmpeg-devel-leave@ffmpeg.org