-
Notifications
You must be signed in to change notification settings - Fork 32
QueryHandler doesn't really check individual command return status #107
Copy link
Copy link
Open
Labels
BugIndicates unexpected or undesired behaviorsIndicates unexpected or undesired behaviorsDiscussion NeededRequire additional context and discussionRequire additional context and discussionPriority: BacklogTasks that are not pressing, but to be tracked and revisitedTasks that are not pressing, but to be tracked and revisited
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.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
BugIndicates unexpected or undesired behaviorsIndicates unexpected or undesired behaviorsDiscussion NeededRequire additional context and discussionRequire additional context and discussionPriority: BacklogTasks that are not pressing, but to be tracked and revisitedTasks that are not pressing, but to be tracked and revisited