Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support JSON-RPCv2 #477

Closed
stevenroose opened this issue Oct 25, 2017 · 4 comments
Closed

Support JSON-RPCv2 #477

stevenroose opened this issue Oct 25, 2017 · 4 comments

Comments

@stevenroose
Copy link

The tagline of the project says gRPC to JSON proxy. However, "JSON" is just a data format. Somehow a RESTful service is created instead of using an existing JSON-based RPC protocol, i.e. JSON-RPC (v2).

What is the motivation of this decision? Why not support JSON-RPC instead of REST?

@AlekSi
Copy link
Contributor

AlekSi commented Oct 26, 2017

That would be a totally different project.

@stevenroose
Copy link
Author

@AlekSi I realize that. Just curious why REST was chosen? REST (being resource-centric) and RPC are fundamentally different. JSON-RPC defines a JSON & HTTP-based RPC protocol, why wasn't it picked over REST?

@achew22
Copy link
Collaborator

achew22 commented Oct 26, 2017

The decision was to implement the gRPC http spec which is a standard part of gRPC implemented by several projects in the same way (Envoy, grpc-httpjson-transcoding, grpc-jersey, and in the paid product side Cloud Endpoints for gRPC APIs). Deviating from the standard would make us incompatible with a wide swath of the gRPC ecosystem.

@achew22
Copy link
Collaborator

achew22 commented Nov 6, 2017

I'm going to close this since it seems to be stuck. If you have any more issues, please reopen/make a new one.

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

No branches or pull requests

3 participants