Merge pull request #27 from thegushi/fix/py3-portability-fixes

Python 3 portability fixes: convert_to_utf8, is_old_password_format, update notification, interpolation, BSD make, configure, common.c

Author: jaredmauch

Mailman/MTA/Postfix.py

@@ -442,12 +442,12 @@ def checkperms(state):
         if stat and (stat[ST_MODE] & targetmode) != targetmode:
             state.ERRORS += 1
             octmode = oct(stat[ST_MODE])
-            print(C_('%(file)s permissions must be 0664 (got %(octmode)s)'),)
+            print(C_('%(file)s permissions must be 0664 (got %(octmode)s)') % locals())
             if state.FIX:
                 print(C_('(fixing)'))
                 os.chmod(file, stat[ST_MODE] | targetmode)
             else:
-                print
+                print()
         # Make sure the corresponding .db files are owned by the Mailman user.
         # We don't need to check the group ownership of the file, since
         # check_perms checks this itself.
@@ -481,7 +481,7 @@ def checkperms(state):
         if stat and (stat[ST_MODE] & targetmode) != targetmode:
             state.ERRORS += 1
             octmode = oct(stat[ST_MODE])
-            print(C_('%(dbfile)s permissions must be 0664 (got %(octmode)s)'),)
+            print(C_('%(dbfile)s permissions must be 0664 (got %(octmode)s)') % locals())
             if state.FIX:
                 print(C_('(fixing)'))
                 os.chmod(dbfile, stat[ST_MODE] | targetmode)

Mailman/Utils.py

@@ -727,6 +727,11 @@ def hash_password(plaintext):
         binascii.hexlify(dk).decode('ascii'))
 
 
+def is_old_password_format(pw):
+    """Return True if pw is a legacy password that needs upgrading to PBKDF2."""
+    return bool(pw) and not pw.startswith(_PBKDF2_SHA256_PREFIX)
+
+
 def verify_password(response, stored):
     """Verify password against stored hash.
 

bin/check_perms

@@ -141,7 +141,7 @@ def checkwalk(arg, dirname, names):
         octperms = oct(targetperms)
         if S_ISDIR(mode) and (mode & targetperms) != targetperms:
             arg.ERRORS += 1
-            print(C_('directory permissions must be %(octperms)s: %(path)s'),)
+            print(C_('directory permissions must be %(octperms)s: %(path)s') % locals())
             if STATE.FIX:
                 print(C_('(fixing)'))
                 os.chmod(path, mode | targetperms)
@@ -150,7 +150,7 @@ def checkwalk(arg, dirname, names):
         elif os.path.splitext(path)[1] in ('.py', '.pyc', '.pyo'):
             octperms = oct(PYFILEPERMS)
             if mode & PYFILEPERMS != PYFILEPERMS:
-                print(C_('source perms must be %(octperms)s: %(path)s'),)
+                print(C_('source perms must be %(octperms)s: %(path)s') % locals())
                 arg.ERRORS += 1
                 if STATE.FIX:
                     print(C_('(fixing)'))
@@ -161,7 +161,7 @@ def checkwalk(arg, dirname, names):
             # Article files must be group writeable
             octperms = oct(ARTICLEFILEPERMS)
             if mode & ARTICLEFILEPERMS != ARTICLEFILEPERMS:
-                print(C_('article db files must be %(octperms)s: %(path)s'),)
+                print(C_('article db files must be %(octperms)s: %(path)s') % locals())
                 arg.ERRORS += 1
                 if STATE.FIX:
                     print(C_('(fixing)'))
@@ -187,7 +187,7 @@ def checkall():
             continue
         if (mode & DIRPERMS) != DIRPERMS:
             STATE.ERRORS += 1
-            print(C_('directory must be at least 02775: %(d)s'),)
+            print(C_('directory must be at least 02775: %(d)s') % locals())
             if STATE.FIX:
                 print(C_('(fixing)'))
                 os.chmod(d, mode | DIRPERMS)
@@ -204,7 +204,7 @@ def checkarchives():
     mode = statmode(private)
     if mode & S_IROTH:
         STATE.ERRORS += 1
-        print(C_('%(private)s must not be other-readable'),)
+        print(C_('%(private)s must not be other-readable') % locals())
         if STATE.FIX:
             print(C_('(fixing)'))
             os.chmod(private, mode & ~S_IROTH)
@@ -253,7 +253,7 @@ def checkarchivedbs():
             continue
         if mode & S_IRWXO:
             STATE.ERRORS += 1
-            print(C_('%(dbdir)s "other" perms must be 000'),)
+            print(C_('%(dbdir)s "other" perms must be 000') % locals())
             if STATE.FIX:
                 print(C_('(fixing)'))
                 os.chmod(dbdir, mode & ~S_IRWXO)
@@ -272,7 +272,7 @@ def checkcgi():
         mode = statmode(path)
         if mode & S_IXGRP and not mode & S_ISGID:
             STATE.ERRORS += 1
-            print(C_('%(path)s must be set-gid'),)
+            print(C_('%(path)s must be set-gid') % locals())
             if STATE.FIX:
                 print(C_('(fixing)'))
                 os.chmod(path, mode | S_ISGID)
@@ -286,7 +286,7 @@ def checkmail():
     mode = statmode(wrapper)
     if not mode & S_ISGID:
         STATE.ERRORS += 1
-        print(C_('%(wrapper)s must be set-gid'),)
+        print(C_('%(wrapper)s must be set-gid') % locals())
         if STATE.FIX:
             print(C_('(fixing)'))
             os.chmod(wrapper, mode | S_ISGID)
@@ -346,7 +346,7 @@ def checkdata():
                 continue
             if (mode & targetmode) != targetmode:
                 STATE.ERRORS += 1
-                print(C_('file permissions must be at least 660: %(path)s'),)
+                print(C_('file permissions must be at least 660: %(path)s') % locals())
                 if STATE.FIX:
                     print(C_('(fixing)'))
                     os.chmod(path, mode | targetmode)

bin/newlist

@@ -241,8 +241,7 @@ def main():
 
     # And send the notice to the list owner
     if not quiet and not automate:
-        print('Hit enter to notify %(listname)s owner...'),
-        sys.stdin.readline()
+        input('Hit enter to notify %s owner...' % listname)
     if not quiet:
         siteowner = Utils.get_site_email(mlist.host_name, 'owner')
         text = Utils.maketext(

bin/update

@@ -814,27 +814,27 @@ def send_password_upgrade_notification(mlist):
         listaddr = mlist.GetListEmail()
         siteowner = Utils.get_site_email(mlist.host_name, 'owner')
 
-        text = _("""\
+        text = C_("""\
 This is an automated message from your Mailman installation.
 
-Your mailing list "%(listname)s" (%(listaddr)s) is using an older password 
-hashing format. For security reasons, we have upgraded Mailman to use a more 
+Your mailing list "%(listname)s" (%(listaddr)s) is using an older password
+hashing format. For security reasons, we have upgraded Mailman to use a more
 secure password hashing method (PBKDF2-SHA256 instead of SHA1).
 
 To complete the upgrade, please log in to your list administration page:
 
     %(admin_url)s
 
-Simply logging in with your current list administrator password will 
-automatically upgrade your password to the new secure format. You don't 
-need to change your password - just log in once and the upgrade will happen 
+Simply logging in with your current list administrator password will
+automatically upgrade your password to the new secure format. You don't
+need to change your password - just log in once and the upgrade will happen
 automatically.
 
-This is a one-time upgrade. After you log in, your password will be 
-automatically converted to the new format and you won't need to do anything 
+This is a one-time upgrade. After you log in, your password will be
+automatically converted to the new format and you won't need to do anything
 else.
 
-If you have any questions or concerns, please contact the site administrator 
+If you have any questions or concerns, please contact the site administrator
 at %(siteowner)s.
 
 Thank you for your attention to this important security upgrade.
@@ -844,8 +844,8 @@ Thank you for your attention to this important security upgrade.
             'admin_url': admin_url,
             'siteowner': siteowner,
         }
-        
-        subject = _('Action Required: Password Upgrade for %(listname)s') % {
+
+        subject = C_('Action Required: Password Upgrade for %(listname)s') % {
             'listname': listname
         }