Merge pull request #206 from 0xdabbad00/remove_prints

revert recent bug fix
duo-labs · Nov 8, 2021 · f46d89a · f46d89a
2 parents 53b1242 + 0492e9e
commit f46d89a
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 35 deletions.
diff --git a/parliament/__init__.py b/parliament/__init__.py
@@ -1,7 +1,7 @@
 """
 This library is a linter for AWS IAM policies.
 """
-__version__ = "1.5.1"
+__version__ = "1.5.2"
 
 import fnmatch
 import functools

diff --git a/parliament/policy.py b/parliament/policy.py
@@ -28,9 +28,6 @@ def __init__(self, policy_json, filepath=None, config=None):
         self.config = config if config else {}
 
     def add_finding(self, finding, detail="", location={}):
-        print(type(location))
-        print(location)
-        print(type({}))
         if type(location) == tuple and "jsoncfg.config_classes" in str(
             type(location[1])
         ):
@@ -45,9 +42,6 @@ def add_finding(self, finding, detail="", location={}):
             location_data["lineno"] = jsoncfg.node_location(location).line
             location_data["column"] = jsoncfg.node_location(location).column
             location = location_data
-
-        if type(location) != dict:
-            location = {'filepath': location}
         if "filepath" not in location:
             location["filepath"] = self.filepath
         self._findings.append(Finding(finding, detail, location))

diff --git a/tests/unit/test_usage.py b/tests/unit/test_usage.py