marshall/unmarshal time.Time to *timestamppb.Timestamp fails about protobuf HOT 8 CLOSED

In order to parse JSON encoded according to the standardized proto3 JSON mapping, you have to use protojson. There is no option. The existing standard encoding/json does not allow all the features necessary to be compliant with the standard, and so we only support that encoding via the newer protojson package.

from protobuf.

If the v2 JSON design eventually gets merged into the stdlib, you would be able to do something like:

jsonv2.Marshal(v, jsonv2.MarshalFuncV1(protojson.Marshal))

which would allow you to handled mixed data structures that include both protobuf messages and non-protobuf Go types.

from protobuf.

Attempting to unmarshal a byte stream that was marshaled by (stdlib) package json into a protobuf isn't guaranteed to work, as @puellanivis said. You should use protojson all the way if you have proto message types (type TestPB in your example), or you should use json all the way if you have JSON-annotated structs (type Test in your example).

It's not clear to me how you're running into the case where you have both type Test and type TestPB. What is generating these two messages?

from protobuf.

@aktau look at annotations in TestPB has both json and protobuf for a field thus I don't see why it should be handled "all the way one way or another" as mutually exclusive. Both proto and json should be able to handle the datatype although the problem was some way introduced by a datatype as struct from protobuf, not carefully thought as solution when data mapping this kind of structs

from protobuf.

Yes, TestPB has annotations for both. I assume this stems from before the time when protojson existed, and the need for some backwards compatibility there. Regardless of all that, there is no guaranteed interoperability between stdlin package encoding/json and protojson. With proto structs, such as your TestPB, please use protojson.

Do you have a problem if you only use package protojson?

from protobuf.

I do and I think it would be good if protojson can unmarshall a string datetime into a timetamp format and fail only if the date/time string is blank/nil/invalid

from protobuf.

I'm still not sure what exactly you'd like, in case it is supported. And I'll note again that we do not support unmarshaling encoding/json encoded json values with protojson. Nor do we support unmarshaling into proto types with anything except for the proto, protojson or prototext packages (I believe that's all).

• If you want a time.Time out of your timestampb field, you can use the AsTime() method: https://pkg.go.dev/google.golang.org/protobuf/types/known/timestamppb.
• The timestamp type has a special JSON representation that's unlike other proto types, see https://pkg.go.dev/google.golang.org/protobuf/types/known/timestamppb#hdr-JSON_Mapping. Also see the implementation: https://github.com/protocolbuffers/protobuf-go/blob/5e8da9414946bbfc6337916c4ee1ce1ef8b95e86/encoding/protojson/well_known_types.go#L759
• We do not support go-proto plugin to generate time.Time types instead of *timestamppb.Timestamp, if that's what you're asking for.

from protobuf.

@aktau, thanks for the support and info provided. Closing this question then.

from protobuf.