Insufficient error description when parsing config

Bug #1621552 reported by Stanislaw Pitucha
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Bandit
Fix Released
Medium
Luke Hinds

Bug Description

Config parsing errors result in:

    [main] ERROR config : Error parsing file.

with no further explanation. Bandit should propagate the information from yaml error, so that the type/line/column of the error are included.

Changed in bandit:
importance: Undecided → Medium
Luke Hinds (lhinds)
Changed in bandit:
assignee: nobody → Luke Hinds (lhinds)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to bandit (master)

Fix proposed to branch: master
Review: https://review.openstack.org/497549

Changed in bandit:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to bandit (master)

Reviewed: https://review.openstack.org/497549
Committed: https://git.openstack.org/cgit/openstack/bandit/commit/?id=05c52da55980bda5358cc337878e27839d3fc6ba
Submitter: Jenkins
Branch: master

commit 05c52da55980bda5358cc337878e27839d3fc6ba
Author: lhinds <email address hidden>
Date: Thu Aug 24 19:32:38 2017 +0100

    Adds simple handler to provide failed line numbers

    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

Changed in bandit:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.