matttproud/golang_protobuf_extensions

Required field "Coordinates.{Unknown}" not set

AlbinOS opened this issue · 1 comments

Dear @matttproud,

I hope you are well since #12 !

We are just experiencing issue with the unmarshal of the same flow and proto file mentioned in the issue above. Here is the issue raised by the pbutil.ReadDelimited function: proto: required field "Coordinates.{Unknown}" not set

Something weird is that, according to the data provider, nothing has changed on the proto file. Even weirder, some message are marshalable and some aren't.

My guess is an issue on the emitter side (since they are updating their stuffs and we didn't touch the receiver code since 1 year) but it's hard to know what's going on.

Maybe you have some clues on what's happening as you might have experienced the same kind of errors while developing this lib ? I'd say that the way they build their frames is not consistent and subject to EOF prematurely before being fully constructed.

Thanks a lot for your work on this lib and it would be amazing if you have any idea of what's going on.

We received notification from the sender that they have issue where one field is omitted in certain case.

In the proto, this field is marked as required and therefore causes the issue. It's just a shame that the proto error is not explicit on what's not set...