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

added GrcpGatewayLogger that implements grpc-middleware #39

Merged
merged 3 commits into from
Jun 14, 2021

Conversation

ogimenezb
Copy link
Contributor

Example included in readme.

It should be good to go.

@ogimenezb ogimenezb changed the title "added GrcpGatewayLogger that implements grpc-middleware" added GrcpGatewayLogger that implements grpc-middleware Jun 14, 2021
@ogimenezb
Copy link
Contributor Author

Once you have revised it and committed can you please tag it so I can make pull request to grpc-ecosystem

@phuslu
Copy link
Owner

phuslu commented Jun 14, 2021

Once you have revised it and committed can you please tag it so I can make pull request to grpc-ecosystem

Sure

@phuslu phuslu merged commit 22df5ca into phuslu:master Jun 14, 2021
@ogimenezb ogimenezb deleted the grpcmiddelware branch June 14, 2021 08:21
@ogimenezb
Copy link
Contributor Author

Ready to make pull request to grpc-ecosystem, let me know once you create a new tag so I can make a clean request with a correct go.mod

@phuslu
Copy link
Owner

phuslu commented Jun 14, 2021

sorry for busy state today, I will tag it ASAP

@ogimenezb
Copy link
Contributor Author

no problem, we all have busy days, take your time.

@phuslu
Copy link
Owner

phuslu commented Jun 14, 2021

I tagged v1.0.72, thanks for your help and patient.

@ogimenezb
Copy link
Contributor Author

Thanks!!!
Made the pull request: grpc-ecosystem/go-grpc-middleware#425
Hope CLA is OK

@ogimenezb
Copy link
Contributor Author

I has been merged as a new log provider go-grpc-middleware for v2.

@phuslu
Copy link
Owner

phuslu commented Jun 20, 2021

oh, many thanks!

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.

2 participants