curl-users
RE : Re: Re : --tcp-keep* options (was Re: [PATCH] SOCKS4a support)
Date: Sat, 12 Jan 2008 19:38:53 +0100 (CET)
Hi,
> I think --tcp-keep-idle should imply --tcp-keep-intvl to the same
> value. So that you set them to the same value with only one command line option
> for it.
>
> Let's do it like this: if the set_so_keepalive() function fails to set
> KEEPALIVE then you won't have to even try to set the TCP_KEEP* options,
> but if it does succeeds with the KEEPALIVE but fails with the TCP_KEEP* we
> should probably output a warning.
>
> BTW, set_so_keepalive is then also better renamed to sockoptcallback or
> similar.
Here is the corresponding patch (code and doc) applied against current CVS, adding a unique --tcp-keepidle option modifying the TCP_KEEPIDLE and TCP_KEEPINTVL options on a per-socket basis.
Sincerely,
Eric LANDES
---------------------------------
Ne gardez plus qu'une seule adresse mail ! Copiez vos mails vers Yahoo! Mail
- text/x-patch attachment: 377219661-keepalive.diff