QueryHandler doesn't really check individual command return status
Open
Bug
Discussion Needed
Priority: Backlog
- Dominant language
- C++
- Stars
- 93
- Forks
- 34
- PR merge metrics
- No merged PRs in 30d
Description
Line 398 in src/QueryHandler.cc which checks for "status" entry in response JSON seems to essentially be a no-op since it does not descend into any command structure. You have to index into the data structure with the command name if you actually want to check status per command.
Contributor guide
Assessment
This issue has not been assessed yet.