[logging] disable option does not work with request/response fields about go-grpc-middleware HOT 6 CLOSED

I'd consider it a separate issue, if you don't mind. If we could tie it to the existing option that would be great too.

from go-grpc-middleware.

Hi. You need to use one of the constants as options: https://github.com/grpc-ecosystem/go-grpc-middleware/pull/631/files#diff-8c12c7fac52459b44598f215bb359e03c4ac3678de743b90e4a88c9ad9b03272R42-R47. It isn't a general purpose log disabling option. We could definitely do with some better documentation here. CC @coleenquadros

from go-grpc-middleware.

Some better docs have been added to this method, hopefully that will avoid confusion around this in the future.

from go-grpc-middleware.

What would be the recommended way to accomplish what I'm trying to accomplish?

from go-grpc-middleware.

It's not possible today AFAIK, but maybe it could be added if it makes sense? I often recommend users just fork if they have very specialized needs, but if you think it makes sense as a general purpose option we could consider that too.

from go-grpc-middleware.

Good to know. I think this use case would be pretty common, "i want to know every time a message is sent but I don't want to log the request or response as it could contain PII"

Would we re-open this as a feature request or a new issue?

from go-grpc-middleware.