openstack/bandit
Code Issues Proposed changes

Adds simple handler to provide failed line numbers

Browse Source 
Change adds `as err` handler to provide line number, if config yaml
fails to parse.

Example output

[config]  ERROR   while scanning a simple key
  in "config.yaml", line 5, column 1
could not find expected ':'
  in "config.yaml", line 6, column 1
[main]  ERROR   config.yaml : Error parsing file.

Change-Id: If764a123c0dd8871dcd98f58be48c6bf0034f1d4
Closes-Bug: #1621552
This commit is contained in:
lhinds 2017-08-24 19:32:38 +01:00
parent d54a65f0f0
commit 05c52da559
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
bandit/core/config.py
View File

@ -49,7 +49,8 @@ class BanditConfig(object):
try: try:
self._config = yaml.safe_load(f) self._config = yaml.safe_load(f)
self.validate(config_file) self.validate(config_file)
except yaml.YAMLError: except yaml.YAMLError as err:
LOG.error(err)
raise utils.ConfigError("Error parsing file.", config_file) raise utils.ConfigError("Error parsing file.", config_file)
# valid config must be a dict # valid config must be a dict