Skip to content
This repository has been archived by the owner on May 17, 2023. It is now read-only.

Support other well known types. e.g. StringValue, Int64Value #29

Open
dpup opened this issue Sep 4, 2018 · 0 comments
Open

Support other well known types. e.g. StringValue, Int64Value #29

dpup opened this issue Sep 4, 2018 · 0 comments

Comments

@dpup
Copy link

dpup commented Sep 4, 2018

These protos are useful for representing optional values where you want to disambiguate empty from nil.

I'm happy to work on a PR. The question is what expected behavior should be.

Given:

message M {
  StringValue str = 1;
}

JSON serializes to:

{}
{"str": ""}
{"str": "some value"}

The first two probably shouldn't be equivalent.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant