Add flake8, coverage, and prepare for release

Author: vkurup

.travis.yml

@@ -35,6 +35,8 @@ env:
   - TOXENV=py36-1.10
   - TOXENV=py36-1.11
   - TOXENV=py36-2.0
+  - TOXENV=flake8
+  - TOXENV=coverage
 
 install:
   - pip install tox

AUTHORS

@@ -2,11 +2,13 @@ The primary development of this project is sponsored by Caktus Consulting Group.
 Thank you to the contributors below which have given time and effort to develop
 and maintain this project.
 
-Johannes Kirschner
-Mark Lavin
-Tobias McNulty
-Alvin Chow
-Vinod Kurup
-Gavin Wahl
+Johannes Kirschner (@jkirschner42)
+Mark Lavin (@mlavin)
+Tobias McNulty (@tobiasmcnulty)
+Alvin Chow (@alvinchow86)
+Vinod Kurup (@vkurup)
+Gavin Wahl (@gavinwahl)
 Arne de Laat (@153957)
 Jaap Roes (@jaap3)
+Tony S Yu (@tonysyu)
+Dan Poirier (@dpoirier)

bandit/__init__.py

@@ -22,4 +22,5 @@ def get_version():
         vers.append('%(releaselevel)s' % __version_info__)
     return ''.join(vers)
 
+
 __version__ = get_version()

bandit/backends/base.py

@@ -3,7 +3,6 @@
 import logging
 
 from email.utils import parseaddr
-from functools import reduce
 
 from django.conf import settings
 from django.template.loader import render_to_string
@@ -13,8 +12,11 @@
 
 class HijackBackendMixin(object):
     """
-    This backend mixin intercepts outgoing messages drops them to a single
-    email address.
+    This backend mixin intercepts outgoing messages, redirecting them to addresses in the
+    BANDIT_EMAIL setting.
+
+    Addresses which are not in ADMINS, SERVER_EMAIL, BANDIT_EMAIL or BANDIT_WHITELIST are
+    intercepted.
     """
 
     def __init__(self, *args, **kwargs):
@@ -49,7 +51,7 @@ def is_approved(email):
                            'previous_recipients': message.to,
                            'previous_cc': message.cc,
                            'previous_bcc': message.bcc
-                           }  # included for backwards compatibility
+                           }  # previous_* values included for backwards compatibility
                 log_message = render_to_string("bandit/hijacked-email-log-message.txt", context)
                 logger.log(self.log_level, log_message)
                 if not self.log_only:

bandit/backends/smtp.py

@@ -7,8 +7,11 @@
 
 class HijackSMTPBackend(HijackBackendMixin, SMTPBackend):
     """
-    This backend intercepts outgoing messages drops them to a single email
-    address.
+    This backend intercepts outgoing messages and logs them, allowing
+    only messages destined for ADMINS, BANDIT_EMAIL, SERVER_EMAIL, or
+    BANDIT_WHITELIST to be sent via SMTP.
+
+    It also sends intercepted messages to BANDIT_EMAIL.
     """
     pass
 
@@ -18,5 +21,7 @@ class LogOnlySMTPBackend(LogOnlyBackendMixin, SMTPBackend):
     This backend intercepts outgoing messages and logs them, allowing
     only messages destined for ADMINS, BANDIT_EMAIL, SERVER_EMAIL, or
     BANDIT_WHITELIST to be sent via SMTP.
+
+    It does not forward intercepted messages.
     """
     pass

setup.cfg

@@ -0,0 +1,2 @@
+[flake8]
+max-line-length = 130

tox.ini

@@ -1,6 +1,8 @@
 [tox]
 envlist = {py27,py35,py36}-{1.8,1.10,1.11},
           {py35,py36}-{2.0},
+          flake8,
+          coverage,
           docs
 
 [testenv]
@@ -15,6 +17,20 @@ deps =
     1.11: Django>=1.11,<2.0
     2.0: Django>=2.0,<2.1
 
+[testenv:flake8]
+basepython = python3.6
+deps = flake8>=3.5,<3.6
+commands = flake8 bandit
+
+[testenv:coverage]
+basepython = python3.6
+commands =
+  coverage run runtests.py
+  coverage report -m --fail-under 85
+deps =
+  Django>=2.0,<2.1
+  coverage>=4.4,<4.5
+
 [testenv:docs]
 basepython = python3.6
 deps =