Serializing protobufs to json #351

Activity

1. dcodeIO commented on Nov 18, 2015

   

   dcodeIO

   on Nov 18, 2015

   Member

   More actions

   Have you tried Message#encodeJSON?

   😕React with 😕2Reacted by T. Coutlakis and Madhav V Rajeev
2. dcodeIO

   added

   question

   on Nov 19, 2015
3. lanttu commented on Dec 16, 2015

   

   lanttu

   on Dec 16, 2015

   More actions

   The method doesn't seem to use canonical proto3 JSON encoding as specfied in here: https://developers.google.com/protocol-buffers/docs/proto3?hl=en#json
4. joeblew99 commented on Dec 17, 2015

   

   joeblew99

   on Dec 17, 2015

   More actions

   Did you sort out if Google's support for json, makes bufferjs redundant ?

   I suspect that bufferjs allows marshalling between json and PB client side, and grpc allows marshalling between the two server side at the facade.

   I would like to know what the answer to this is from an architectural perspective.
5. leeyeh commented on Feb 23, 2016

   

   leeyeh

   on Feb 23, 2016

   More actions

   +1.

   My use case is to encode to and decode from the canonical formatted JSON data, since React Native's js-native bright is text-based.
6. dcodeIO commented on Nov 28, 2016

   

   dcodeIO

   on Nov 28, 2016

   Member

   More actions

   Closing this for now.

   protobuf.js 6.0.0

   Feel free to send a pull request if this is still a requirement.
7. dcodeIO

   closed this as completedon Nov 28, 2016
8. aidansteele commented on Apr 11, 2017

   

   aidansteele

   on Apr 11, 2017

   More actions

   @dcodeIO Sorry, it's not clear to me - is the issue closed because any changes discussed here are no longer relevant (due to rewrite) or because protobuf.js 6.x now supports the functionality?
9. dcodeIO commented on Apr 11, 2017

   

   dcodeIO

   on Apr 11, 2017

   Last edited by dcodeIO

   Member

   More actions

   or because protobuf.js 6.x now supports the functionality

   Not quite, but the default toJSON implementation of message instances should be close.

   someMessage.toJSON();
   // or
   SomeMessage.toObject(someMessage, util.toJSONOptions);

   Not yet supported:

   • "NaN", "Infinity" strings for floats
   • Any, Timestamp, Duration, Struct, Wrappers, FieldMask, ListValue, Value, NullValue are handled like messages

   Feel free to reopen!
10. dcodeIO

    added a commit that references this issue on Apr 11, 2017

    New: Added 'json' conversion option for proto3 JSON mapping compatibi…

    ...

    3939667
11. dcodeIO commented on Apr 11, 2017

    

    dcodeIO

    on Apr 11, 2017

    Member

    More actions

    Related: #677