[Date Prev][Date Next]
Re: tcp timeoutcon patch for libldap
At 06:56 PM 7/28/99 +0200, Lars Uffmann wrote:
>This sounds like we should look for an I/O
>abstraction layer in llber.
Yes, at the base BER level there should be a very
simple abstraction layer. read/write should be enough...
>For my particular problem,
>the posibility to provide a custom connect() function
>via ldap_set_option, would have solved the problem.
Yes, I believe there should be an additional layer
of hooks at the LDAP API level.
>> >To set/get the timeout, I used
>> >the yet unused LDAP_OPT_TIMEOUT option flag
>> >in ldap.h.
>> The LDAP_OPT_TIMEOUT was meant to be used as a
>> per LDAP API call timeout versus a per network call
>> timeout. Implementation of such gets a little
>Per API call timeouts is a differrent story ...
>I think the API specs are quite inconsistent. For example,
>there is no ldap_bind_st() api function.
>But we solved this by writing a custom bind_st() function.
Or use a client control with with ldap_*_ext_s().
The _st() routines, in my option, should be deprecated.
>The only issue left is the bind call when following
>referrals. Do you have any suggestions how to get arround this?
If a API timeout was specified, the time spent obtaining
the referral must be substracted from timeout used
in the chasing request.
>Maybee LDAP_OPT_TIMEOUT could be used here.
Actually, my point about using controls brings up an
another possibility. A default client control could
(and maybe should) be used instead of LDAP_OPT_TIMEOUT.
>> If you want a per network call timeout, I would
>> suggest introducing another option.
>If you are willing to include my patch / provide the
>timeoutconnect feature (either way) I would be happy
This, too, could be handled by a client control...
(but an opt is fine for now).
>> Note: The invalue of these options should be
>> "struct timeval *".
>I can correct this, if you think it is important.
For now, can you resubmit (as an ITS) using
LDAP_OPT_NETWORK_TIMEOUT and struct timeval *
(be sure duplicate the structure in and out of
We'll also should to sort out configure issues
surrounding inet_aton() and O_NDELAY...