Broken Redpath about libredfish HOT 4 CLOSED

It does look like you're using this correctly. @pboyd04 any ideas?

from libredfish.

I'm not sure if it ever worked, because I can't find any parsing code for the [parent.child].

In redpath.c:165 the propname just gets set to index. (Status.Health in my example) with the REDPATH_OP_EXISTS operation.
Then later, when this operation gets executed in payload.c this same path string gets used (as nodeName) when jansson is called: value = json_object_get(payload->json, nodeName)
Only this function doesn't support json paths, only json keys.

I have two ideas on how I might be able to fix this:

1. Add a json_object_get_by_path function and use it in payload.c which would be able to handle these json paths.
2. Make redpath.c split this [parent.child] into two seperate redpathnodes that get handled separately.

IMHO option 1 is way simpler, and I already have working code that can handle json paths. I think I can implement it in payload.c, without breaking any current functionality. But I would have to test this first.

Let me know what you think about it.

from libredfish.

Please see #163

from libredfish.

Fix merged

from libredfish.