-
Notifications
You must be signed in to change notification settings - Fork 33
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
Add middleware for logging requests/responses via the grpc-web-devtools #442
Add middleware for logging requests/responses via the grpc-web-devtools #442
Conversation
I'm sorry, I didn't have enough time and energy lately to review this, but I didn't forget about it! |
Thank you for you contribution @segfault16! Please address these comments 👆 |
…t16/nice-grpc into grpc-devtools-middleware
@aikoven Comments have been addressed and I've added some tests for ts-proto and grpc-js |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job! Looks mostly done to me. Please address last couple of comments:
Thanks! I'm going to merge this and release shortly. |
Published 📦 Could not figure out how to correctly publish a |
resolves #27.
This is an initial version for the middleware.
I've copied the structure in ``nice-grpc-client-middleware-deadline` and added the logic used in my project.
Feedback and help welcome ;)