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.

Sign up for GitHub

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

Jump to bottom

Support gRPC Reflection API #448

Closed
cvanderschuere opened this issue Jun 12, 2020 · 5 comments
Closed

Support gRPC Reflection API #448

cvanderschuere opened this issue Jun 12, 2020 · 5 comments
Assignees
@alexshtin
Labels
enhancement New feature or request needs-investigation P2

Comments

@cvanderschuere
Copy link

Is your feature request related to a problem? Please describe.
Exploring APIs requires linking to the protobuf definitions. This makes using tools such as grpcurl or grpcui more difficult

Describe the solution you'd like
Support gRPC Reflection for Frontend APIs.

Additional context

Unfortunately, this is not currently possible without a much more significant change of protobuf libraries. There is more context in this blog post, but the short is that without this change there will be errors similar to:

$ grpcui  --plaintext localhost:7233 
Failed to compute set of methods to expose: Symbol not found: adminservice.AdminService
@cvanderschuere cvanderschuere added the enhancement New feature or request label Jun 12, 2020
@samarabbas
Copy link
Contributor

@alexshtin can you look into this?

@alexshtin
Copy link
Member

alexshtin commented Jul 13, 2021 •
edited
Loading

gRPC reflection will come automatically if we do #38. It doesn't work with gogo implementation. The only APIs that supported are health check APIs (which served by google generated code).

@alexshtin
Copy link
Member

Btw, https://github.com/ktr0731/evans is another great tool to explore gRPC services.

@yiminc yiminc added P1 P2 and removed P1 labels Oct 29, 2022
@sync-by-unito sync-by-unito bot closed this as completed Mar 3, 2023
@yiminc yiminc reopened this Mar 3, 2023
@alexshtin
Copy link
Member

This was essentially enabled by #5032.

@alexshtin
Copy link
Member

Can be used with evans:

echo '{"namespace":"temporal-system"}' | evans --host 127.0.0.1 --port 7233 --reflection cli call temporal.api.workflowservice.v1.WorkflowService.ListWorkflowExecutions

or
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alexshtin alexshtin

Labels
enhancement New feature or request needs-investigation P2
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@cvanderschuere @samarabbas @alexshtin @yiminc