xmidt-org / wrp-go

go implementation of the Web Routing Protocol
Apache License 2.0
4 stars 7 forks source link

Strict ugorji encoding #50

Open johnabass opened 4 years ago

johnabass commented 4 years ago

We currently configure the ugorji library to be very loose when decoding. For example, spans was defined as a [][]string, but when the underlying message format changed there was no error because ugorji silently did the conversion.

Should we continue to do this? Do we want XMiDT cluster code to throw an error when a msgpack or JSON message is encountered that does not strictly conform to the WRP spec?