From e626540dfb470386d0750f2f8bbaf4b5cb36b203 Mon Sep 17 00:00:00 2001 From: Andrey Smirnov Date: Thu, 8 Sep 2022 14:18:39 +0400 Subject: [PATCH] chore: avoid double API request logging in trustd There's a common logger for API calls already working, so no need to log in the token authenticator. Signed-off-by: Andrey Smirnov --- pkg/grpc/middleware/auth/basic/token.go | 14 +------------- 1 file changed, 1 insertion(+), 13 deletions(-) diff --git a/pkg/grpc/middleware/auth/basic/token.go b/pkg/grpc/middleware/auth/basic/token.go index 6e4f96e9e..00d3f319f 100644 --- a/pkg/grpc/middleware/auth/basic/token.go +++ b/pkg/grpc/middleware/auth/basic/token.go @@ -7,8 +7,6 @@ package basic import ( "context" "fmt" - "log" - "time" "google.golang.org/grpc" "google.golang.org/grpc/codes" @@ -81,20 +79,10 @@ func (b *TokenCredentials) authenticate(ctx context.Context) error { // basic authentication. func (b *TokenCredentials) UnaryInterceptor() grpc.UnaryServerInterceptor { return func(ctx context.Context, req interface{}, info *grpc.UnaryServerInfo, handler grpc.UnaryHandler) (interface{}, error) { - start := time.Now() - if err := b.authenticate(ctx); err != nil { return nil, err } - h, err := handler(ctx, req) - - log.Printf("request - Method:%s\tDuration:%s\tError:%v\n", - info.FullMethod, - time.Since(start), - err, - ) - - return h, err + return handler(ctx, req) } }