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

fix reflect proto #574

Closed
wants to merge 1 commit into from
Closed

fix reflect proto #574

wants to merge 1 commit into from

Conversation

tonybase
Copy link

@tonybase tonybase commented May 31, 2019

  • fix register name to use import path.
  • register to golang proto.
  • resolve grpc_cli reflection.
  • resolve protoreflect library.
  • resolve bloomrpc GUI client for gRPC.

use gofast or gogofast generate api.proto:

$ ./grpc_cli ls 127.0.0.1:9000 demo.service.v1.Demo -l
filename: demo-grpc/api/api.proto
package: demo.service.v1;
service Demo {
  rpc SayHello(demo.service.v1.HelloReq) returns (google.protobuf.Empty) {}
  rpc SayHelloURL(demo.service.v1.HelloReq) returns (demo.service.v1.HelloResp) {}
}

@tonybase tonybase changed the title fix golang proto fix reflection proto Jun 1, 2019
@tonybase tonybase changed the title fix reflection proto fix reflect proto Jun 1, 2019
@prestonvanloon
Copy link

What can we do to get this merged? We're also facing the same problem.

@daniel-cohen
Copy link

I am willing to help fix the conflicts if it helps gets this merged. I am not sure what to do about the broken reflection otherwise.

@tonybase tonybase closed this Dec 5, 2020
@daniel-cohen
Copy link

@tonybase What did you end up doing ?

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

Successfully merging this pull request may close these issues.

None yet

3 participants