fix(grpc_error): made sure to return 401 in case of errutil.ErrorReasonUnauthorized

Signed-off-by: Mohamed Tahar KEDJOUR <mkedjour@cisco.com>

Author: mkedjour

backend/internal/pkg/grpcutil/errores_test.go

@@ -35,6 +35,10 @@ func TestGrpcError(t *testing.T) {
 			inErr:  errutil.InvalidRequest("c", "c"),
 			outErr: grpcutil.BadRequestError(errutil.InvalidRequest("c", "c")),
 		},
+		"should return unauthorized status error for ErrorReasonUnauthorized": {
+			inErr:  errutil.Unauthorized("d", "d"),
+			outErr: grpcutil.UnauthorizedError(errutil.Unauthorized("d", "d")),
+		},
 		"should return the same error as input when it's not DomainError": {
 			inErr:  generalErr,
 			outErr: generalErr,

backend/internal/pkg/grpcutil/errors.go

@@ -39,6 +39,8 @@ func Error(err error) error {
 			return BadRequestError(err)
 		case errutil.ErrorReasonInvalidRequest:
 			return BadRequestError(err)
+		case errutil.ErrorReasonUnauthorized:
+			return UnauthorizedError(err)
 		}
 	}
 

backend/internal/pkg/interceptors/contextual_logger.go

@@ -15,14 +15,7 @@ import (
 	"google.golang.org/grpc/metadata"
 )
 
-// Enriches logs with identity context (tenantID, userID, etc.)
-// and information about the current gRPC request.
-//
-// It does this by adding a value containing those fields to the context
-// and pass it to the next handler, and to include those fields when logging,
-// use log.FromContext(ctx).
-
-// Enriches the context with tenant and request metadata for structured logging.
+// ContextualLoggerUnary enriches the context with tenant and request metadata for structured logging.
 //
 // This interceptor extracts tenant related information (such as tenant ID, app ID, etc.) from
 // the incoming context, as well as the full gRPC method name and the current request.