From e5a0a3998dbc44bfada055f3867634c84e4bede0 Mon Sep 17 00:00:00 2001
From: ghubstan <36207203+ghubstan@users.noreply.github.com>
Date: Sat, 27 Feb 2021 17:14:22 -0300
Subject: [PATCH] Permit some gRPC excptions to be logged only as warning

A new handleExceptionAsWarning method logs warn(ex.msg) instead of
the full stack trace.
---
 .../bisq/daemon/grpc/GrpcExceptionHandler.java    | 15 ++++++++++++++-
 1 file changed, 14 insertions(+), 1 deletion(-)

diff --git a/daemon/src/main/java/bisq/daemon/grpc/GrpcExceptionHandler.java b/daemon/src/main/java/bisq/daemon/grpc/GrpcExceptionHandler.java
index 38665474e9..5ab763aafa 100644
--- a/daemon/src/main/java/bisq/daemon/grpc/GrpcExceptionHandler.java
+++ b/daemon/src/main/java/bisq/daemon/grpc/GrpcExceptionHandler.java
@@ -46,7 +46,9 @@ class GrpcExceptionHandler {
     public GrpcExceptionHandler() {
     }
 
-    public void handleException(Logger log, Throwable t, StreamObserver<?> responseObserver) {
+    public void handleException(Logger log,
+                                Throwable t,
+                                StreamObserver<?> responseObserver) {
         // Log the core api error (this is last chance to do that), wrap it in a new
         // gRPC StatusRuntimeException, then send it to the client in the gRPC response.
         log.error("", t);
@@ -55,6 +57,17 @@ class GrpcExceptionHandler {
         throw grpcStatusRuntimeException;
     }
 
+    public void handleExceptionAsWarning(Logger log,
+                                        String calledMethod,
+                                        Throwable t,
+                                        StreamObserver<?> responseObserver) {
+        // Just log a warning instead of an error with full stack trace.
+        log.warn(calledMethod + " -> " + t.getMessage());
+        var grpcStatusRuntimeException = wrapException(t);
+        responseObserver.onError(grpcStatusRuntimeException);
+        throw grpcStatusRuntimeException;
+    }
+
     private StatusRuntimeException wrapException(Throwable t) {
         // We want to be careful about what kinds of exception messages we send to the
         // client.  Expected core exceptions should be wrapped in an IllegalStateException