that's great!

On Thu, Dec 15, 2011 at 12:43 PM, Omi Chiba <ochib...@gmail.com> wrote:
> Nico,
>
> Yes, it works !
> Actually the code is there for the error check reason so instead of comment
> out, I moved the line 92 - 94  after line 105 where result variables is
> declared.
>
> The attached file is the modified "ldap_auth.py" and I will send the patch
> to Massimo shortly.
>
> Looks like this:
> ---------------------------
> result = con.search_ext_s(
>                     ldap_basedn, ldap.SCOPE_SUBTREE,
>                     "(&(sAMAccountName=%s)(%s))" % (username_bare,
> filterstr), ["sAMAccountName"])[0][1]
>                 if not isinstance(result, dict):
>                     # result should be a dict in the form {'sAMAccountName':
> [username_bare]}
>                     return False
>

Reply via email to