Re: Extension Refresh (RFC 2589)

On 5 Mar 2010, at 15:53, Etienne Bagnoud wrote:

> Hi,
> 
> I needed the extended operation "refresh" as Net::LDAP extension. As I
> didn't find one, I wrote one.
> This is mainly copied from SetPassword.pm, it's my first Perl module I
> wrote.
> The copyright notice is set that way as I didn't know what to set
> precisely, let me know if I'm wrong and it could be corrected.
> 
> The code is available for download (~3K)
> http://www.tchetch.net/code/perl/Refresh.pm.txt

It looks fine to me, though I'm not familiar with that RFC.

You should also add an OID to Net::LDAP::Constant so that callers can easily test for this extension's presence in the root DSE.

Cheers,

Chris