Log DebugException from RpcException if present (#1625)

Author: JamesNK

src/Grpc.AspNetCore.Server/Internal/GrpcServerLog.cs

@@ -129,9 +129,9 @@ public static void ErrorExecutingServiceMethod(ILogger logger, string serviceMet
             _errorExecutingServiceMethod(logger, serviceMethod, ex);
         }
 
-        public static void RpcConnectionError(ILogger logger, StatusCode statusCode, string detail)
+        public static void RpcConnectionError(ILogger logger, StatusCode statusCode, string detail, Exception? debugException)
         {
-            _rpcConnectionError(logger, statusCode, detail, null);
+            _rpcConnectionError(logger, statusCode, detail, debugException);
         }
 
         public static void EncodingNotInAcceptEncoding(ILogger logger, string grpcEncoding)

src/Grpc.AspNetCore.Server/Internal/HttpContextServerCallContext.cs

@@ -179,8 +179,9 @@ private void ProcessHandlerError(Exception ex, string method)
             if (ex is RpcException rpcException)
             {
                 // RpcException is thrown by client code to modify the status returned from the server.
-                // Log the status and detail. Don't log the exception to reduce log verbosity.
-                GrpcServerLog.RpcConnectionError(Logger, rpcException.StatusCode, rpcException.Status.Detail);
+                // Log the status, detail and debug exception (if present).
+                // Don't log the RpcException itself to reduce log verbosity. All of its information is already captured.
+                GrpcServerLog.RpcConnectionError(Logger, rpcException.StatusCode, rpcException.Status.Detail, rpcException.Status.DebugException);
 
                 // There are two sources of metadata entries on the server-side:
                 // 1. serverCallContext.ResponseTrailers