Fixed! thanks everybody.
On Dec 15, 7:46 pm, Nik Go <[email protected]> wrote: > Everytime a *new* stable version is out, I come to expect minor releases > happening in the next several days and try to hold off my own upgrade for > as long as possible. :P > > > > > > > > On Friday, December 16, 2011, Massimo Di Pierro wrote: > > Looks like we need a 1.99.5 thanks Omi > > > On Dec 15, 9:43 am, Omi Chiba <[email protected] <javascript:;>> 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 > > > > ldap_auth.py > > > 7KViewDownload
