Ignore some static-analysis false positives

abadger · Apr 3, 2019 · 80ecf8e · 80ecf8e
1 parent 8ae498f
commit 80ecf8e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/magnate/dispatchers/account.py b/magnate/dispatchers/account.py
@@ -47,7 +47,8 @@
 
 def login(username, password):
     flog = mlog.fields(func='login')
-    flog.fields(username=username, password='*OMITTED*').debug('Entered login')
+    # Bandit thinks that *OMITTED* is the password instead of non-logging of the password
+    flog.fields(username=username, password='*OMITTED*').debug('Entered login')  # nosec
 
     try:
         magnate.login(username, password)

diff --git a/tests/units/test_auth.py b/tests/units/test_auth.py
@@ -1,8 +1,10 @@
+# The idiom for unittests is to use classes for organization
+# pylint: disable=no-self-use
+
 import dbm
 import os
 import os.path
 import stat
-from collections.abc import MutableMapping
 
 import pytest
 from passlib.hash import pbkdf2_sha512