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

Logging #92

Closed
bufdev opened this issue Jan 20, 2016 · 7 comments
Closed

Logging #92

bufdev opened this issue Jan 20, 2016 · 7 comments

Comments

@bufdev
Copy link
Contributor

bufdev commented Jan 20, 2016

Hey,

Old topic, simpler idea: I don't want to re-invent the wheel, but can we do something similar to https://github.com/grpc/grpc-go/tree/master/grpclog for gprc-gateway? Using glog means everyone is locked into glog's very opinionated way of storing output, and it means in a lot of cases (including mine) that I just completely lose grpc-gateway logs.

The only change in the code would be to get rid of v-level logging, but I mean this is minor as I see it within the code.

@yugui
Copy link
Member

yugui commented Jan 21, 2016

I agree on its benefit. But I prefer simply using grpclog package.
What do you think?

@dmitshur
Copy link
Contributor

Wasn't there a very similar proposal before? See #54. Or is this different?

@yugui
Copy link
Member

yugui commented Jan 21, 2016

I don't remember why I closed #54. But maybe I confused it with adding dependency to dlog?

I did not want to add extra dependency for that small improvement. But grpclog is free from that concern.

@bufdev
Copy link
Contributor Author

bufdev commented Jan 21, 2016

Fine with me!

@bufdev
Copy link
Contributor Author

bufdev commented Jan 21, 2016

I can put together a PR

@crast
Copy link

crast commented Feb 14, 2016

Any reason not to simply use the package google.golang.org/grpc/grpclog directly? There's already a hard dependency on google.golang.org/grpc and this package is always at risk of dealing with the API changes in GRPC, since it directly imports over.

I would assert that people who want to set the logging output for GRPC probably don't necessarily need to set different logging output for GRPC gateway - they generally either want it going somewhere they can control it, or they want it fully off

@yugui
Copy link
Member

yugui commented Feb 15, 2016

@crast You are right. I am fine with grpclog.

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

No branches or pull requests

4 participants