--On Wednesday, April 03, 2013 12:56 PM -0500 Graham Barr <gb...@pobox.com> wrote:

but keepalive is not inherently part of LDAP any more than it is part of
SMTP or FTP or any other protocol.

OTOH, making long term, *persistent* connections IS a part of the LDAP protocol, and it is NOT generally something one does with SMTP or FTP. You are comparing apples to oranges.

I see this as no different than the OpenLDAP C API having already added support for the extended keepalive parameters ages ago when it is possible, or otherwise silently ignoring them. It is part of the *nature* of LDAP to allow, and *encourage* persistent connections, many of which remain idle for long periods of time.

But this implementation gives the impression that it is and may surprised
someone when it silently does not do what is expected because of the host
they ran it on.

No, this implementation allows the use of it IF it is available. My patch specifically documents the cases in which it is. This is why documentation is provided to the end user.

I.e., it doesn't in any way make Net::LDAP platform specific,

No, it adds non-ldap specific implementaiton

See above.  This is something that is *very* specific to the LDAP protocol.

--Quanah


--

Quanah Gibson-Mount
Sr. Member of Technical Staff
Zimbra, Inc
A Division of VMware, Inc.
--------------------
Zimbra ::  the leader in open source messaging and collaboration

Reply via email to