Don't catch exceptions in service methods

It makes handling exceptions in interceptors hard. On the other hand, if there is no interceptor catching StatusRuntimeException the response status is UNKNOWN.
SuduIDE · Oct 19, 2023 · 44c62b8 · 44c62b8
1 parent 81efd60
commit 44c62b8
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/...or/src/main/java/org/sudu/protogen/generator/server/OverriddenServiceMethodGenerator.java b/...or/src/main/java/org/sudu/protogen/generator/server/OverriddenServiceMethodGenerator.java
@@ -61,11 +61,11 @@ private TypeModel responseTypeModel() {
     @NotNull
     private CodeBlock generateBody() {
         return CodeBlock.of("""
-                try {
-                $>$L$<
-                } catch (Throwable $$t) { responseObserver.onError($$t); }
-                finally { responseObserver.onCompleted(); }
-                """, generateAbstractMethodCall());
+                        $L
+                        responseObserver.onCompleted();
+                        """,
+                generateAbstractMethodCall()
+        );
     }
 
     private CodeBlock generateAbstractMethodCall() {