omit bare except.

active_bots/mastodonbot.py

@@ -21,7 +21,7 @@ class MastodonBot(Bot):
         m = Mastodon(*user.get_masto_credentials())
         try:
             notifications = m.notifications()
-        except:  # mastodon.Mastodon.MastodonAPIError is unfortunately not in __init__.py
+        except Exception:
             logger.error("Unknown Mastodon API Error.", exc_info=True)
             return mentions
         for status in notifications:

backend.py

@@ -4,7 +4,7 @@ import active_bots
 from config import config
 from db import db
 import logging
-import sendmail
+from sendmail import sendmail
 import time
 
 
@@ -30,12 +30,9 @@ if __name__ == '__main__':
                         for bot2 in bots:
                             bot2.post(user, status)
             time.sleep(60)  # twitter rate limit >.<
-    except:
+    except Exception:
-        logger.error('Shutdown', exc_info=True)
+        logger.error('Shutdown.', exc_info=True)
-        mailer = sendmail.Mailer()
     try:
-            mailer.send('', config['web']['contact'],
+        sendmail(config['web']['contact'], 'Ticketfrei Shutdown')
-                        'Ticketfrei Crash Report',
+    except Exception:
-                        attachment=config['logging']['logpath'])
+        logger.error('Could not inform admin.', exc_info=True)
-        except:
-            logger.error('Mail sending failed', exc_info=True)

frontend.py

@@ -150,7 +150,7 @@ def login_mastodon(user):
         return dict(
                 info='Thanks for supporting decentralized social networks!'
             )
-    except:
+    except Exception:
         logger.error('Login to Mastodon failed.', exc_info=True)
         return dict(error='Login to Mastodon failed.')
 

sendmail.py

@@ -64,7 +64,7 @@ class Mailer(object):
         return "Sent mail to " + recipient + ": " + subject
 
 
-def sendmail(to, subject, body):
+def sendmail(to, subject, body=''):
     msg = MIMEMultipart()
     msg['From'] = '%s@%s' % (getuser(), getfqdn())
     msg['To'] = to