The parser should not fail on malformed yaml

Bug #1599598 reported by Joe Talbott
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Snapcraft
Fix Released
High
Joe Talbott

Bug Description

The parser should not fail on malformed yaml but should log a warning.

An errant entry in the wiki caused the parser to fail and CI to fail due to parser integration test failures.

Joe Talbott (joetalbott)
Changed in snapcraft:
assignee: nobody → Joe Talbott (joetalbott)
Joe Talbott (joetalbott)
Changed in snapcraft:
status: New → In Progress
Revision history for this message
Joe Talbott (joetalbott) wrote :

Add a fake server to represent the wiki in the fix.

Revision history for this message
Sergio Schvezov (sergiusens) wrote :
Changed in snapcraft:
milestone: none → 2.13
importance: Undecided → High
status: In Progress → Fix Committed
Revision history for this message
Seth Arnold (seth-arnold) wrote :

Continuing on in the face of errors can lead to very confusing situations. e.g. someone may try changing a setting from true to false to 1 to 0 to yes to no when trying to make something work but if the parser doesn't report e.g. that the setting _name_ is unexpected the real problem may be harder to find.

I'd rather use tools that die with clear error messages any day...

Thanks

Changed in snapcraft:
status: Fix Committed → 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.