-
-
Notifications
You must be signed in to change notification settings - Fork 78
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
Document grpc-dump output format #11
Comments
I think it would be useful to also have some sample jq expressions for people to get started with. I can help out with that if you'd like. :) |
This would be great 🙂 I was thinking about having a full walkthrough using something like https://grpcweb.jbrandhorst.com/ as an example so this would fit in very nicely. What example expressions were you thinking of? Off the top of my head maybe we could have: filtering for requests that returned an error, filtering for requests for a specific service/method, and some sort of counting/summarisation? |
Things like filtering for metadata, the client response, the server, first element of a repeated series. I will make a tiny pr tonight and throw it in for some help. If there are other things that you are looking to add and want some help i'd be keen, i wanted to build something similar to what you did here for quite some time now actually 👍 |
Very happy to accept help. In the short term I'm going to do some hardening against misconfiguration e.g. this TODO: grpc-tools/grpc-proxy/grpc_handler.go Line 39 in 147eef4
But feature-wise, adding support for the gRPC reflection API would be really useful. Also looking for some good ideas on how to solve #6 but it seems quite a tricky one... |
Yes, actually i have been looking into the reflection api in the past couple of weeks for some things at work. Happy to go deeper on that :) About #6 give me a bit of time to get the context. |
Without a proper specification of the dump output format it's hard for users to write parsers, etc. for the output of
grpc-dump
.The text was updated successfully, but these errors were encountered: