From 83fca4c614c4bd561b369749821676bba5ce467a Mon Sep 17 00:00:00 2001 From: Pablo Vega Date: Fri, 26 Jan 2018 00:28:39 -0800 Subject: [PATCH] Adding a check to make sure either spaces or null values are not mistakenly added into the key variable --- pre_commit_hooks/detect_aws_credentials.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/pre_commit_hooks/detect_aws_credentials.py b/pre_commit_hooks/detect_aws_credentials.py index b2afd71..47a69bf 100644 --- a/pre_commit_hooks/detect_aws_credentials.py +++ b/pre_commit_hooks/detect_aws_credentials.py @@ -53,7 +53,9 @@ def get_aws_secrets_from_file(credentials_file): 'aws_session_token', ): try: - keys.add(parser.get(section, var)) + key = parser.get(section, var).strip() + if key: + keys.add(key) except configparser.NoOptionError: pass return keys