Skip to content

[CONFIGURATION] Handle invalid yaml #3857

@marcalff

Description

@marcalff

This is a follow up on #3851 .

The fix in 3851 is valid and addresses immediate needs, but there is more to the story.

A follow up PR is needed to:

  • add a unit test in sdk/test/configuration/yaml_test.cc
  • install ryml error handlers explicitly, instead of relying on the default behavior, by installing Callbacks in ryml

See https://github.com/biojppm/rapidyaml/blob/c1d0047fd3f30b09133487ba5bb74195816b02e7/src/c4/yml/common.hpp#L510

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workinggood first issueGood for newcomershelp wantedGood for taking. Extra help will be provided by maintainerstriage/acceptedIndicates an issue or PR is ready to be actively worked on.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions