[Freeipa-devel] [PATCH] 956 user lockout status

Simo Sorce simo at redhat.com
Tue Feb 21 22:28:16 UTC 2012


On Tue, 2012-02-21 at 15:57 -0500, Rob Crittenden wrote:
> +                other_ldap = ldap2(shared_instance=False,
> +                                   ldap_uri='ldap://%s' % host,
> +                                   base_dn=self.api.env.basedn)
> +                other_ldap.connect(ccache=os.environ['KRB5CCNAME']) 

Nack,
if a server is not reachable this will give back a traceback as no
exception is trapped.
Also in general we should not fail, some servers may have whatever
issues. We should just return a "N/A" result for those servers that we
had problem reaching or requesting values from.

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York




More information about the Freeipa-devel mailing list