Skip to content

yaml: be more robust when parsing objects vs sequences#394

Open
MariusBgm wants to merge 4 commits into
mainfrom
fix/yaml_parse_healthcheck
Open

yaml: be more robust when parsing objects vs sequences#394
MariusBgm wants to merge 4 commits into
mainfrom
fix/yaml_parse_healthcheck

Conversation

@MariusBgm

Copy link
Copy Markdown
Collaborator

This makes the parsing of objects more robust.
the following was silently accepted:

HealthCheck:   
  - SoftResponseTimeout: 1

which should be invalid, but is silently accepted (because HealthCheck is a map and not a container for a sequence).

Signed-off-by: Marius Börschig <Marius.Boerschig@vector.com>
Signed-off-by: Marius Börschig <Marius.Boerschig@vector.com>
@MariusBgm MariusBgm changed the title Fix/yaml parse healthcheck yaml: be more robust when parsing objects vs sequences Jul 23, 2026
Comment thread SilKit/source/config/YamlReader.hpp
Comment thread SilKit/source/config/YamlReader.hpp
Reject multiple document streams

Signed-off-by: Marius Börschig <Marius.Boerschig@vector.com>
…uences

allow only single document streams, document it and update changelog

Signed-off-by: Marius Börschig <Marius.Boerschig@vector.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants