userdir_ldap.py: pycodestyle fixes
authorJulien Cristau <jcristau@debian.org>
Fri, 11 Oct 2019 16:26:42 +0000 (18:26 +0200)
committerJulien Cristau <jcristau@debian.org>
Fri, 11 Oct 2019 16:26:42 +0000 (18:26 +0200)
userdir_ldap.py

index 22722e4..05a19bf 100644 (file)
@@ -35,9 +35,9 @@ import termios
 import userdir_gpg
 
 try:
-   File = open("/etc/userdir-ldap/userdir-ldap.conf")
+    File = open("/etc/userdir-ldap/userdir-ldap.conf")
 except IOError:
-   File = open("userdir-ldap.conf")
+    File = open("userdir-ldap.conf")
 ConfModule = imp.load_source("userdir_config", "/etc/userdir-ldap.conf", File)
 File.close()
 
@@ -57,19 +57,19 @@ Ech_MainLog = ConfModule.ech_mainlog
 HostDomain = getattr(ConfModule, "hostdomain", EmailAppend)
 
 try:
-   UseSSL = ConfModule.usessl
+    UseSSL = ConfModule.usessl
 except AttributeError:
-   UseSSL = False
+    UseSSL = False
 
 try:
-   BaseBaseDn = ConfModule.basebasedn
+    BaseBaseDn = ConfModule.basebasedn
 except AttributeError:
-   BaseBaseDn = BaseDn
+    BaseBaseDn = BaseDn
 
 try:
-   IgnoreUsersForUIDNumberGen = ConfModule.ignoreusersforuidnumbergen
+    IgnoreUsersForUIDNumberGen = ConfModule.ignoreusersforuidnumbergen
 except AttributeError:
-   IgnoreUsersForUIDNumberGen = ['nobody']
+    IgnoreUsersForUIDNumberGen = ['nobody']
 
 
 # Break up the keyring list
@@ -92,395 +92,395 @@ GroupObjectClasses = ("top", "debianGroup")
 
 # SSH Key splitting. The result is:
 # (options,size,modulous,exponent,comment)
-SSHAuthSplit = re.compile('^(.* )?(\d+) (\d+) (\d+) ?(.+)$')
-SSH2AuthSplit = re.compile('^(.* )?ssh-(dss|rsa|ecdsa-sha2-nistp(?:256|384|521)|ed25519) ([a-zA-Z0-9=/+]+) ?(.+)$')
-AddressSplit = re.compile("(.*).*<([^@]*)@([^>]*)>")
+SSHAuthSplit = re.compile(r'^(.* )?(\d+) (\d+) (\d+) ?(.+)$')
+SSH2AuthSplit = re.compile(r'^(.* )?ssh-(dss|rsa|ecdsa-sha2-nistp(?:256|384|521)|ed25519) ([a-zA-Z0-9=/+]+) ?(.+)$')
+AddressSplit = re.compile(r"(.*).*<([^@]*)@([^>]*)>")
 
 
 # Safely get an attribute from a tuple representing a dn and an attribute
 # list. It returns the first attribute if there are multi.
 def GetAttr(DnRecord, Attribute, Default=""):
-   try:
-      return DnRecord[1][Attribute][0]
-   except IndexError:
-      return Default
-   except KeyError:
-      return Default
-   return Default
+    try:
+        return DnRecord[1][Attribute][0]
+    except IndexError:
+        return Default
+    except KeyError:
+        return Default
+    return Default
 
 
 # Return a printable email address from the attributes.
 def EmailAddress(DnRecord):
-   cn = GetAttr(DnRecord, "cn")
-   sn = GetAttr(DnRecord, "sn")
-   uid = GetAttr(DnRecord, "uid")
-   if cn == "" and sn == "":
-      return "<" + uid + "@" + EmailAppend + ">"
-   return cn + " " + sn + " <" + uid + "@" + EmailAppend + ">"
+    cn = GetAttr(DnRecord, "cn")
+    sn = GetAttr(DnRecord, "sn")
+    uid = GetAttr(DnRecord, "uid")
+    if cn == "" and sn == "":
+        return "<" + uid + "@" + EmailAppend + ">"
+    return cn + " " + sn + " <" + uid + "@" + EmailAppend + ">"
 
 
 # Show a dump like ldapsearch
 def PrettyShow(DnRecord):
-   Result = ""
-   List = DnRecord[1].keys()
-   List.sort()
-   for x in List:
-      Rec = DnRecord[1][x]
-      for i in Rec:
-         Result = Result + "%s: %s\n" % (x, i)
-   return Result[:-1]
+    Result = ""
+    List = DnRecord[1].keys()
+    List.sort()
+    for x in List:
+        Rec = DnRecord[1][x]
+        for i in Rec:
+            Result = Result + "%s: %s\n" % (x, i)
+    return Result[:-1]
 
 
 def connectLDAP(server=None):
-   if server is None:
-      global LDAPServer
-      server = LDAPServer
-   lc = ldap.initialize('ldap://%s' % server)
-   global UseSSL
-   if UseSSL:
-      lc.start_tls_s()
-   return lc
+    if server is None:
+        global LDAPServer
+        server = LDAPServer
+    lc = ldap.initialize('ldap://%s' % server)
+    global UseSSL
+    if UseSSL:
+        lc.start_tls_s()
+    return lc
 
 
 def passwdAccessLDAP(BaseDn, AdminUser):
-   """
-   Ask for the AdminUser's password and connect to the LDAP server.
-   Returns the connection handle.
-   """
-   print "Accessing LDAP directory as '" + AdminUser + "'"
-   while (1):
-      if 'LDAP_PASSWORD' in os.environ:
-          Password = os.environ['LDAP_PASSWORD']
-      else:
-          Password = getpass.getpass(AdminUser + "'s password: ")
-
-      if len(Password) == 0:
-         sys.exit(0)
-
-      lc = connectLDAP()
-      UserDn = "uid=" + AdminUser + "," + BaseDn
-
-      # Connect to the ldap server
-      try:
-         lc.simple_bind_s(UserDn, Password)
-      except ldap.INVALID_CREDENTIALS:
-         if 'LDAP_PASSWORD' in os.environ:
-             print "password in environment does not work"
-             del os.environ['LDAP_PASSWORD']
-         continue
-      break
-   return lc
+    """
+    Ask for the AdminUser's password and connect to the LDAP server.
+    Returns the connection handle.
+    """
+    print "Accessing LDAP directory as '" + AdminUser + "'"
+    while (1):
+        if 'LDAP_PASSWORD' in os.environ:
+            Password = os.environ['LDAP_PASSWORD']
+        else:
+            Password = getpass.getpass(AdminUser + "'s password: ")
+
+        if len(Password) == 0:
+            sys.exit(0)
+
+        lc = connectLDAP()
+        UserDn = "uid=" + AdminUser + "," + BaseDn
+
+        # Connect to the ldap server
+        try:
+            lc.simple_bind_s(UserDn, Password)
+        except ldap.INVALID_CREDENTIALS:
+            if 'LDAP_PASSWORD' in os.environ:
+                print "password in environment does not work"
+                del os.environ['LDAP_PASSWORD']
+            continue
+        break
+    return lc
 
 
 # Split up a name into multiple components. This tries to best guess how
 # to split up a name
 def NameSplit(Name):
-   Words = re.split(" ", Name.strip())
+    Words = re.split(" ", Name.strip())
 
-   # Insert an empty middle name
-   if len(Words) == 2:
-      Words.insert(1, "")
-   if len(Words) < 2:
-      Words.append("")
+    # Insert an empty middle name
+    if len(Words) == 2:
+        Words.insert(1, "")
+    if len(Words) < 2:
+        Words.append("")
 
-   # Put a dot after any 1 letter words, must be an initial
-   for x in range(0, len(Words)):
-      if len(Words[x]) == 1:
-         Words[x] = Words[x] + '.'
+    # Put a dot after any 1 letter words, must be an initial
+    for x in range(0, len(Words)):
+        if len(Words[x]) == 1:
+            Words[x] = Words[x] + '.'
 
-   # If a word starts with a -, ( or [ we assume it marks the start of some
-   # Non-name information and remove the remainder of the string
-   for x in range(0, len(Words)):
-      if len(Words[x]) != 0 and (Words[x][0] == '-' or
-                                 Words[x][0] == '(' or Words[x][0] == '['):
-         Words = Words[0:x]
-         break
+    # If a word starts with a -, ( or [ we assume it marks the start of some
+    # Non-name information and remove the remainder of the string
+    for x in range(0, len(Words)):
+        if len(Words[x]) != 0 and (Words[x][0] == '-' or
+                                   Words[x][0] == '(' or Words[x][0] == '['):
+            Words = Words[0:x]
+            break
 
-   # Merge any of the middle initials
-   while len(Words) > 2 and len(Words[2]) == 2 and Words[2][1] == '.':
-      Words[1] = Words[1] + Words[2]
-      del Words[2]
+    # Merge any of the middle initials
+    while len(Words) > 2 and len(Words[2]) == 2 and Words[2][1] == '.':
+        Words[1] = Words[1] + Words[2]
+        del Words[2]
 
-   while len(Words) < 2:
-      Words.append('')
+    while len(Words) < 2:
+        Words.append('')
 
-   # Merge any of the last name prefixes into one big last name
-   while Words[-2].lower() in LastNamesPre:
-      Words[-1] = Words[-2] + " " + Words[-1]
-      del Words[-2]
+    # Merge any of the last name prefixes into one big last name
+    while Words[-2].lower() in LastNamesPre:
+        Words[-1] = Words[-2] + " " + Words[-1]
+        del Words[-2]
 
-   # Fix up a missing middle name after lastname globbing
-   if len(Words) == 2:
-      Words.insert(1, "")
+    # Fix up a missing middle name after lastname globbing
+    if len(Words) == 2:
+        Words.insert(1, "")
 
-   # If the name is multi-word then we glob them all into the last name and
-   # do not worry about a middle name
-   if (len(Words) > 3):
-      Words[2] = " ".join(Words[1:])
-      Words[1] = ""
+    # If the name is multi-word then we glob them all into the last name and
+    # do not worry about a middle name
+    if (len(Words) > 3):
+        Words[2] = " ".join(Words[1:])
+        Words[1] = ""
 
-   return (Words[0].strip(), Words[1].strip(), Words[2].strip())
+    return (Words[0].strip(), Words[1].strip(), Words[2].strip())
 
 
 # Compute a random password using /dev/urandom
 def GenPass():
-   # Generate a 10 character random string
-   SaltVals = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ/."
-   Rand = open("/dev/urandom")
-   Password = ""
-   for i in range(0, 15):
-      Password = Password + SaltVals[ord(Rand.read(1)[0]) % len(SaltVals)]
-   return Password
+    # Generate a 10 character random string
+    SaltVals = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ/."
+    Rand = open("/dev/urandom")
+    Password = ""
+    for i in range(0, 15):
+        Password = Password + SaltVals[ord(Rand.read(1)[0]) % len(SaltVals)]
+    return Password
 
 
 # Compute the MD5 crypted version of the given password
 def HashPass(Password):
-   # Hash it telling glibc to use the MD5 algorithm - if you dont have
-   # glibc then just change Salt = "$1$" to Salt = "";
-   SaltVals = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789/."
-   Salt = "$1$"
-   Rand = open("/dev/urandom")
-   for x in range(0, 10):
-      Salt = Salt + SaltVals[ord(Rand.read(1)[0]) % len(SaltVals)]
-   Pass = crypt.crypt(Password, Salt)
-   if len(Pass) < 14:
-      raise Exception("MD5 password hashing failed, not changing the password!")
-   return Pass
+    # Hash it telling glibc to use the MD5 algorithm - if you dont have
+    # glibc then just change Salt = "$1$" to Salt = "";
+    SaltVals = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789/."
+    Salt = "$1$"
+    Rand = open("/dev/urandom")
+    for x in range(0, 10):
+        Salt = Salt + SaltVals[ord(Rand.read(1)[0]) % len(SaltVals)]
+    Pass = crypt.crypt(Password, Salt)
+    if len(Pass) < 14:
+        raise Exception("MD5 password hashing failed, not changing the password!")
+    return Pass
 
 
 # Sync with the server, we count the number of async requests that are pending
 # and make sure result has been called that number of times
 def FlushOutstanding(l, Outstanding, Fast=0):
-   # Sync with the remote end
-   if Fast == 0:
-      print "Waiting for {} requests:".format(Outstanding),
-   while (Outstanding > 0):
-      try:
-         if Fast == 0 or Outstanding > 50:
-            sys.stdout.write(".",)
-            sys.stdout.flush()
-            if (l.result(ldap.RES_ANY, 1) != (None, None)):
-               Outstanding = Outstanding - 1
-         else:
-            if (l.result(ldap.RES_ANY, 1, 0) != (None, None)):
-               Outstanding = Outstanding - 1
+    # Sync with the remote end
+    if Fast == 0:
+        print "Waiting for {} requests:".format(Outstanding),
+    while (Outstanding > 0):
+        try:
+            if Fast == 0 or Outstanding > 50:
+                sys.stdout.write(".",)
+                sys.stdout.flush()
+                if (l.result(ldap.RES_ANY, 1) != (None, None)):
+                    Outstanding = Outstanding - 1
             else:
-               break
-      except ldap.TYPE_OR_VALUE_EXISTS:
-         Outstanding -= 1
-      except ldap.NO_SUCH_ATTRIBUTE:
-         Outstanding -= 1
-      except ldap.NO_SUCH_OBJECT:
-         Outstanding -= 1
-   if Fast == 0:
-      print
-   return Outstanding
+                if (l.result(ldap.RES_ANY, 1, 0) != (None, None)):
+                    Outstanding = Outstanding - 1
+                else:
+                    break
+        except ldap.TYPE_OR_VALUE_EXISTS:
+            Outstanding -= 1
+        except ldap.NO_SUCH_ATTRIBUTE:
+            Outstanding -= 1
+        except ldap.NO_SUCH_OBJECT:
+            Outstanding -= 1
+    if Fast == 0:
+        print
+    return Outstanding
 
 
 # Convert a lat/long attribute into Decimal degrees
 def DecDegree(Posn, Anon=0):
-  Parts = re.match('[-+]?(\d*)\\.?(\d*)', Posn).groups()
-  Val = float(Posn)
-
-  if (abs(Val) >= 1806060.0):
-      raise ValueError("Too Big")
-
-  # Val is in DGMS
-  if abs(Val) >= 18060.0 or len(Parts[0]) > 5:
-      Val = Val / 100.0
-      Secs = Val - long(Val)
-      Val = long(Val) / 100.0
-      Min = Val - long(Val)
-      Val = long(Val) + (Min * 100.0 + Secs * 100.0 / 60.0) / 60.0
-
-  # Val is in DGM
-  elif abs(Val) >= 180 or len(Parts[0]) > 3:
-     Val = Val / 100.0
-     Min = Val - long(Val)
-     Val = long(Val) + Min * 100.0 / 60.0
-
-  if Anon != 0:
-      Str = "%3.2f" % (Val)
-  else:
-      Str = str(Val)
-  if Val >= 0:
-     return "+" + Str
-  return Str
+    Parts = re.match(r'[-+]?(\d*)\\.?(\d*)', Posn).groups()
+    Val = float(Posn)
+
+    if (abs(Val) >= 1806060.0):
+        raise ValueError("Too Big")
+
+    # Val is in DGMS
+    if abs(Val) >= 18060.0 or len(Parts[0]) > 5:
+        Val = Val / 100.0
+        Secs = Val - long(Val)
+        Val = long(Val) / 100.0
+        Min = Val - long(Val)
+        Val = long(Val) + (Min * 100.0 + Secs * 100.0 / 60.0) / 60.0
+
+    # Val is in DGM
+    elif abs(Val) >= 180 or len(Parts[0]) > 3:
+        Val = Val / 100.0
+        Min = Val - long(Val)
+        Val = long(Val) + Min * 100.0 / 60.0
+
+    if Anon != 0:
+        Str = "%3.2f" % (Val)
+    else:
+        Str = str(Val)
+    if Val >= 0:
+        return "+" + Str
+    return Str
 
 
 def FormatSSH2Auth(Str):
-   Match = SSH2AuthSplit.match(Str)
-   if Match is None:
-      return "<unknown format>"
-   G = Match.groups()
+    Match = SSH2AuthSplit.match(Str)
+    if Match is None:
+        return "<unknown format>"
+    G = Match.groups()
 
-   if G[0] is None:
-      return "ssh-%s %s..%s %s" % (G[1], G[2][:8], G[2][-8:], G[3])
-   return "%s ssh-%s %s..%s %s" % (G[0], G[1], G[2][:8], G[2][-8:], G[3])
+    if G[0] is None:
+        return "ssh-%s %s..%s %s" % (G[1], G[2][:8], G[2][-8:], G[3])
+    return "%s ssh-%s %s..%s %s" % (G[0], G[1], G[2][:8], G[2][-8:], G[3])
 
 
 def FormatSSHAuth(Str):
-   Match = SSHAuthSplit.match(Str)
-   if Match is None:
-      return FormatSSH2Auth(Str)
-   G = Match.groups()
+    Match = SSHAuthSplit.match(Str)
+    if Match is None:
+        return FormatSSH2Auth(Str)
+    G = Match.groups()
 
-   # No options
-   if G[0] is None:
-      return "%s %s %s..%s %s" % (G[1], G[2], G[3][:8], G[3][-8:], G[4])
-   return "%s %s %s %s..%s %s" % (G[0], G[1], G[2], G[3][:8], G[3][-8:], G[4])
+    # No options
+    if G[0] is None:
+        return "%s %s %s..%s %s" % (G[1], G[2], G[3][:8], G[3][-8:], G[4])
+    return "%s %s %s %s..%s %s" % (G[0], G[1], G[2], G[3][:8], G[3][-8:], G[4])
 
 
 def FormatPGPKey(Str):
-   Res = ""
-
-   # PGP 2.x Print
-   if (len(Str) == 32):
-      indent = 0
-      while (indent < len(Str)):
-         if indent == 32 / 2:
-            Res = "%s %s%s " % (Res, Str[indent], Str[indent + 1])
-         else:
-            Res = "%s%s%s " % (Res, Str[indent], Str[indent + 1])
-         indent += 2
-   elif (len(Str) == 40):
-      # OpenPGP Print
-      indent = 0
-      while (indent < len(Str)):
-         if indent == 40 / 2:
-            Res = "%s %s%s%s%s " % (Res, Str[indent], Str[indent + 1], Str[indent + 2], Str[indent + 3])
-         else:
-            Res = "%s%s%s%s%s " % (Res, Str[indent], Str[indent + 1], Str[indent + 2], Str[indent + 3])
-         indent += 4
-   else:
-      Res = Str
-   return Res.strip()
+    Res = ""
+
+    # PGP 2.x Print
+    if (len(Str) == 32):
+        indent = 0
+        while (indent < len(Str)):
+            if indent == 32 / 2:
+                Res = "%s %s%s " % (Res, Str[indent], Str[indent + 1])
+            else:
+                Res = "%s%s%s " % (Res, Str[indent], Str[indent + 1])
+            indent += 2
+    elif (len(Str) == 40):
+        # OpenPGP Print
+        indent = 0
+        while (indent < len(Str)):
+            if indent == 40 / 2:
+                Res = "%s %s%s%s%s " % (Res, Str[indent], Str[indent + 1], Str[indent + 2], Str[indent + 3])
+            else:
+                Res = "%s%s%s%s%s " % (Res, Str[indent], Str[indent + 1], Str[indent + 2], Str[indent + 3])
+            indent += 4
+    else:
+        Res = Str
+    return Res.strip()
 
 
 # Take an email address and split it into 3 parts, (Name,UID,Domain)
 def SplitEmail(Addr):
-   # Is not an email address at all
-   if Addr.find('@') == -1:
-      return (Addr, "", "")
+    # Is not an email address at all
+    if Addr.find('@') == -1:
+        return (Addr, "", "")
 
-   Res1 = rfc822.AddrlistClass(Addr).getaddress()
-   if len(Res1) != 1:
-      return ("", "", Addr)
-   Res1 = Res1[0]
-   if Res1[1] is None:
-      return (Res1[0], "", "")
+    Res1 = rfc822.AddrlistClass(Addr).getaddress()
+    if len(Res1) != 1:
+        return ("", "", Addr)
+    Res1 = Res1[0]
+    if Res1[1] is None:
+        return (Res1[0], "", "")
 
-   # If there is no @ then the address was not parsed well. Try the alternate
-   # Parsing scheme. This is particularly important when scanning PGP keys.
-   Res2 = Res1[1].split("@")
-   if len(Res2) != 2:
-      Match = AddressSplit.match(Addr)
-      if Match is None:
-         return ("", "", Addr)
-      return Match.groups()
+    # If there is no @ then the address was not parsed well. Try the alternate
+    # Parsing scheme. This is particularly important when scanning PGP keys.
+    Res2 = Res1[1].split("@")
+    if len(Res2) != 2:
+        Match = AddressSplit.match(Addr)
+        if Match is None:
+            return ("", "", Addr)
+        return Match.groups()
 
-   return (Res1[0], Res2[0], Res2[1])
+    return (Res1[0], Res2[0], Res2[1])
 
 
 # Convert the PGP name string to a uid value. The return is a tuple of
 # (uid,[message strings]). UnknownMpa is a hash from email to uid that
 # overrides normal searching.
 def GetUID(l, Name, UnknownMap={}):
-   # Crack up the email address into a best guess first/middle/last name
-   (cn, mn, sn) = NameSplit(re.sub('["]', '', Name[0]))
-
-   # Brackets anger the ldap searcher
-   cn = re.sub('[(")]', '?', cn)
-   sn = re.sub('[(")]', '?', sn)
-
-   # First check the unknown map for the email address
-   if Name[1] + '@' + Name[2] in UnknownMap:
-      Stat = "unknown map hit for " + str(Name)
-      return (UnknownMap[Name[1] + '@' + Name[2]], [Stat])
-
-   # Then the cruft component (ie there was no email address to match)
-   if Name[2] in UnknownMap:
-      Stat = "unknown map hit for" + str(Name)
-      return (UnknownMap[Name[2]], [Stat])
-
-   # Then the name component (another ie there was no email address to match)
-   if Name[0] in UnknownMap:
-      Stat = "unknown map hit for" + str(Name)
-      return (UnknownMap[Name[0]], [Stat])
-
-   # Search for a possible first/last name hit
-   try:
-      Attrs = l.search_s(BaseDn, ldap.SCOPE_ONELEVEL, "(&(cn=%s)(sn=%s))" % (cn, sn), ["uid"])
-   except ldap.FILTER_ERROR:
-      Stat = "Filter failure: (&(cn=%s)(sn=%s))" % (cn, sn)
-      return (None, [Stat])
-
-   # Try matching on the email address
-   if len(Attrs) != 1:
-      try:
-         Attrs = l.search_s(BaseDn, ldap.SCOPE_ONELEVEL, "emailforward=%s" % (Name[2]), ["uid"])
-      except ldap.FILTER_ERROR:
-          pass
-
-   # Hmm, more than one/no return
-   if len(Attrs) != 1:
-      # Key claims a local address
-      if Name[2] == EmailAppend:
-
-         # Pull out the record for the claimed user
-         Attrs = l.search_s(BaseDn, ldap.SCOPE_ONELEVEL, "(uid=%s)" % (Name[1]), ["uid", "sn", "cn"])
-
-         # We require the UID surname to be someplace in the key name, this
-         # deals with special purpose keys like 'James Troup (Alternate Debian key)'
-         # Some people put their names backwards on their key too.. check that as well
-         if len(Attrs) == 1 and \
-            (sn.lower().find(Attrs[0][1]["sn"][0].lower()) != -1 or
-            cn.lower().find(Attrs[0][1]["sn"][0].lower()) != -1):
-            Stat = EmailAppend + " hit for " + str(Name)
-            return (Name[1], [Stat])
-
-      # Attempt to give some best guess suggestions for use in editing the
-      # override file.
-      Attrs = l.search_s(BaseDn, ldap.SCOPE_ONELEVEL, "(sn~=%s)" % (sn,), ["uid", "sn", "cn"])
-
-      Stat = []
-      if len(Attrs) != 0:
-         Stat = ["None for %s" % (str(Name))]
-      for x in Attrs:
-         Stat.append("But might be: %s %s <%s@debian.org>" % (x[1]["cn"][0], x[1]["sn"][0], x[1]["uid"][0]))
-      return (None, Stat)
-   else:
-      return (Attrs[0][1]["uid"][0], None)
-
-   return (None, None)
+    # Crack up the email address into a best guess first/middle/last name
+    (cn, mn, sn) = NameSplit(re.sub('["]', '', Name[0]))
+
+    # Brackets anger the ldap searcher
+    cn = re.sub('[(")]', '?', cn)
+    sn = re.sub('[(")]', '?', sn)
+
+    # First check the unknown map for the email address
+    if Name[1] + '@' + Name[2] in UnknownMap:
+        Stat = "unknown map hit for " + str(Name)
+        return (UnknownMap[Name[1] + '@' + Name[2]], [Stat])
+
+    # Then the cruft component (ie there was no email address to match)
+    if Name[2] in UnknownMap:
+        Stat = "unknown map hit for" + str(Name)
+        return (UnknownMap[Name[2]], [Stat])
+
+    # Then the name component (another ie there was no email address to match)
+    if Name[0] in UnknownMap:
+        Stat = "unknown map hit for" + str(Name)
+        return (UnknownMap[Name[0]], [Stat])
+
+    # Search for a possible first/last name hit
+    try:
+        Attrs = l.search_s(BaseDn, ldap.SCOPE_ONELEVEL, "(&(cn=%s)(sn=%s))" % (cn, sn), ["uid"])
+    except ldap.FILTER_ERROR:
+        Stat = "Filter failure: (&(cn=%s)(sn=%s))" % (cn, sn)
+        return (None, [Stat])
+
+    # Try matching on the email address
+    if len(Attrs) != 1:
+        try:
+            Attrs = l.search_s(BaseDn, ldap.SCOPE_ONELEVEL, "emailforward=%s" % (Name[2]), ["uid"])
+        except ldap.FILTER_ERROR:
+            pass
+
+    # Hmm, more than one/no return
+    if len(Attrs) != 1:
+        # Key claims a local address
+        if Name[2] == EmailAppend:
+
+            # Pull out the record for the claimed user
+            Attrs = l.search_s(BaseDn, ldap.SCOPE_ONELEVEL, "(uid=%s)" % (Name[1]), ["uid", "sn", "cn"])
+
+            # We require the UID surname to be someplace in the key name, this
+            # deals with special purpose keys like 'James Troup (Alternate Debian key)'
+            # Some people put their names backwards on their key too.. check that as well
+            if len(Attrs) == 1 and \
+                (sn.lower().find(Attrs[0][1]["sn"][0].lower()) != -1 or
+                 cn.lower().find(Attrs[0][1]["sn"][0].lower()) != -1):
+                Stat = EmailAppend + " hit for " + str(Name)
+                return (Name[1], [Stat])
+
+        # Attempt to give some best guess suggestions for use in editing the
+        # override file.
+        Attrs = l.search_s(BaseDn, ldap.SCOPE_ONELEVEL, "(sn~=%s)" % (sn,), ["uid", "sn", "cn"])
+
+        Stat = []
+        if len(Attrs) != 0:
+            Stat = ["None for %s" % (str(Name))]
+        for x in Attrs:
+            Stat.append("But might be: %s %s <%s@debian.org>" % (x[1]["cn"][0], x[1]["sn"][0], x[1]["uid"][0]))
+        return (None, Stat)
+    else:
+        return (Attrs[0][1]["uid"][0], None)
+
+    return (None, None)
 
 
 def Group2GID(l, name):
-   """
-   Returns the numerical id of a common group
-   on error returns -1
-   """
-   for g in DebianGroups.keys():
-      if name == g:
-         return DebianGroups[g]
+    """
+    Returns the numerical id of a common group
+    on error returns -1
+    """
+    for g in DebianGroups.keys():
+        if name == g:
+            return DebianGroups[g]
 
-   filter = "(gid=%s)" % name
-   res = l.search_s(BaseDn, ldap.SCOPE_ONELEVEL, filter, ["gidNumber"])
-   if res:
-      return int(GetAttr(res[0], "gidNumber"))
+    filter = "(gid=%s)" % name
+    res = l.search_s(BaseDn, ldap.SCOPE_ONELEVEL, filter, ["gidNumber"])
+    if res:
+        return int(GetAttr(res[0], "gidNumber"))
 
-   return -1
+    return -1
 
 
 def make_hmac(str):
-   if 'UD_HMAC_KEY' in os.environ:
-      HmacKey = os.environ['UD_HMAC_KEY']
-   else:
-      File = open(PassDir + "/key-hmac-" + pwd.getpwuid(os.getuid())[0], "r")
-      HmacKey = File.readline().strip()
-      File.close()
-   return hmac.new(HmacKey, str, hashlib.sha1).hexdigest()
+    if 'UD_HMAC_KEY' in os.environ:
+        HmacKey = os.environ['UD_HMAC_KEY']
+    else:
+        File = open(PassDir + "/key-hmac-" + pwd.getpwuid(os.getuid())[0], "r")
+        HmacKey = File.readline().strip()
+        File.close()
+    return hmac.new(HmacKey, str, hashlib.sha1).hexdigest()
 
 
 def make_passwd_hmac(status, purpose, uid, uuid, hosts, cryptedpass):
-   return make_hmac(':'.join([status, purpose, uid, uuid, hosts, cryptedpass]))
+    return make_hmac(':'.join([status, purpose, uid, uuid, hosts, cryptedpass]))