fix: log user in grpc mw

internal/app/mw/error_type.go

@@ -11,6 +11,8 @@ const (
 	respServerError
 )
 
+const unknownError = "unknown error"
+
 // gRPCRespErrorTypeFromStatusCode returns response error type depending on gRPC status code.
 func gRPCRespErrorTypeFromStatusCode(statusCode codes.Code) respErrorType {
 	switch statusCode {
@@ -36,3 +38,10 @@ func httpRespErrorTypeFromStatusCode(statusCode int) respErrorType {
 		return respServerError
 	}
 }
+
+func processErrorMessage(err string) string {
+	if err == "" {
+		return unknownError
+	}
+	return err
+}

internal/app/mw/grpc_mw.go

@@ -30,12 +30,17 @@ func GRPCLogInterceptor(l *tracing.Logger) grpc.UnaryServerInterceptor {
 		if err != nil {
 			l.Error(ctx, "failed to marshal request message", zap.Error(err))
 		}
+
 		reqLogArgs := requestLogArgs{
 			component:   "gRPC",
 			header:      md,
 			fullMethod:  info.FullMethod,
 			requestBody: string(rBody),
 		}
+		if userName, err := types.GetUserKey(ctx); err == nil {
+			reqLogArgs.user = userName
+		}
+
 		logRequestBeforeHandler(ctx, l, reqLogArgs)
 
 		start := time.Now()
@@ -52,15 +57,13 @@ func GRPCLogInterceptor(l *tracing.Logger) grpc.UnaryServerInterceptor {
 
 		errType := gRPCRespErrorTypeFromStatusCode(st.Code())
 
-		if errType == respClientError {
-			reqLogArgs.clientError = st.Err().Error()
-		}
-		if errType == respServerError {
-			reqLogArgs.serverError = st.Err().Error()
-		}
-		if userName, err := types.GetUserKey(ctx); err == nil {
-			reqLogArgs.user = userName
+		switch errType {
+		case respClientError:
+			reqLogArgs.clientError = processErrorMessage(st.Err().Error())
+		case respServerError:
+			reqLogArgs.serverError = processErrorMessage(st.Err().Error())
 		}
+
 		if details := st.Details(); len(details) > 0 {
 			reqLogArgs.details = details
 		}

internal/app/mw/http_mw.go

@@ -77,16 +77,11 @@ func HTTPLogInterceptor(l *tracing.Logger) func(next http.Handler) http.Handler
 			reqLogArgs.statusCode = http.StatusText(statusCodeInt)
 			reqLogArgs.took = took
 
-			if errType == respClientError { // nolint:staticcheck
-				reqLogArgs.clientError = ww.ErrorMessage
-				if reqLogArgs.clientError == "" {
-					reqLogArgs.clientError = "unknown error"
-				}
-			} else if errType == respServerError {
-				reqLogArgs.serverError = ww.ErrorMessage
-				if reqLogArgs.serverError == "" {
-					reqLogArgs.serverError = "unknown error"
-				}
+			switch errType {
+			case respClientError:
+				reqLogArgs.clientError = processErrorMessage(ww.ErrorMessage)
+			case respServerError:
+				reqLogArgs.serverError = processErrorMessage(ww.ErrorMessage)
 			}
 
 			logRequestAfterHandler(r.Context(), l, reqLogArgs)