↪️ Merge pull request #168 from dgzlopes/fix-117-whitelist-additional…

…-text-support Add support for text after whitelist
Yelp · May 10, 2019 · dd4de73 · dd4de73
2 parents 458d7e9 + 3117bcd
commit dd4de73
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/detect_secrets/plugins/common/constants.py b/detect_secrets/plugins/common/constants.py
@@ -4,7 +4,7 @@
 WHITELIST_REGEXES = [
     re.compile(r)
     for r in [
-        r'[ \t]+{} *pragma: ?whitelist[ -]secret{}[ \t]*$'.format(start, end)
+        r'[ \t]+{} *pragma: ?whitelist[ -]secret.*?{}[ \t]*$'.format(start, end)
         for start, end in (
             ('#', ''),                    # e.g. python or yaml
             ('//', ''),                   # e.g. golang

diff --git a/tests/plugins/high_entropy_strings_test.py b/tests/plugins/high_entropy_strings_test.py
@@ -101,10 +101,12 @@ def test_analyze_multiple_strings_same_line(self, content_to_format, expected_re
         [
             # Test inline annotation for whitelisting
             "'{secret}'  # pragma: whitelist secret",
+            "'{secret}'  # pragma: whitelist secret more text",
             "'{secret}' #  pragma: whitelist secret",
             "'{secret}'  // pragma: whitelist secret",
             "'{secret}' //  pragma: whitelist secret",
             "'{secret}'  /* pragma: whitelist secret */",
+            "'{secret}'  /* pragma: whitelist secret more text */",
             "'{secret}' /*  pragma: whitelist secret  */",
             "'{secret}'  ' pragma: whitelist secret",
             "'{secret}' '  pragma: whitelist secret",