Re: start_tls

[Graham B. <gb...@po...>]

On Fri, Jul 06, 2001 at 10:23:44AM +0100, Chris Ridd wrote:
> I've committed additional changes to LDAPS.pm, LDAP.pm and LDAP.pod. I just
> copied the documentation across because I thought it was still useful to
> see directly on the LDAPS page.

I see you have overridded start_tls in LDAPS with a croak.

I want to avoid croaks if we can. We could just return a sucess
as afterall we are already using SSL.

This check really needs to go into Net::LDAP, as what should happen
if start_tls is called twice ?

I suggest we add a check in start_tls for $sock->isa('IO::Socket::SSL')

Graham.