go-grpc-middleware icon indicating copy to clipboard operation
go-grpc-middleware copied to clipboard

v2: Please provide (port) grpc_logsettable.SettableLoggerV2 in go-grpc-middleware/v2

Open pavankrish123 opened this issue 3 years ago • 2 comments
trafficstars

Unfortunately v2 doesn't have SettableLoggerV2 nor has APIs for setting global gRPC logger, as a matter of fact. Only interceptor logs are available on the branch which will not produce framework level logs.

We rely on SettableLoggerV2 for setting global grpc logger to zap logger and reset it back as needed.

Please let me know if you need any further information.

pavankrish123 avatar Dec 02 '21 17:12 pavankrish123

I was doing some research and found this comment https://github.com/grpc-ecosystem/go-grpc-middleware/pull/402#issue-831821664.

Is setting grpc logger via grpc.SetLoggerV2() not something we do not want in v2 at all?

pavankrish123 avatar Dec 02 '21 21:12 pavankrish123

I would have to defer to @bwplotka who implemented most of the logging changes 😁

johanbrandhorst avatar Dec 03 '21 03:12 johanbrandhorst

I am failing to understand what global grpc Logger has to do with interceptors (even logging interceptor) 🤔

Do you mind giving us TL;DR on how this repo (on v2 branch) is limiting you on using https://pkg.go.dev/google.golang.org/grpc/grpclog#SetLoggerV2 in your code? Thanks 🤗

bwplotka avatar Mar 19 '23 01:03 bwplotka

Closing for now, since we didn't hear from you - we can always open, thanks! (:

bwplotka avatar Apr 13 '23 10:04 bwplotka