gnmic icon indicating copy to clipboard operation
gnmic copied to clipboard

JsonIetfVal of JsonVal in the form a json array might not be processed properly.

Open Tobelightbeam opened this issue 1 year ago • 2 comments

https://github.com/openconfig/gnmic/blob/345c505534afda492e27b7c1afd76cd72ae095e9/formatters/event.go#L260C1-L260C1

JsonVal or JsonIetfVal might be in the form of a json array, but in the code corresponding to the above link, the json array form is classified into the default case. Will this kind of handling cause any problems?

Tobelightbeam avatar Oct 18 '23 09:10 Tobelightbeam

While it's not a common practice for a router to send a value as a list at the top level of a JSON object, it typically sends multiple notifications, each containing a JSON dictionary as a value. However, it might be a good idea to address this potential issue.

karimra avatar Oct 20 '23 11:10 karimra

@karimra thanks for your reply. If you have any plan to do this, please let me know.

Tobelightbeam avatar Oct 23 '23 02:10 Tobelightbeam