Merge pull request #17 from padilo/feature/send_500_in_body

Send string Internal Server Error
scm-spain · Nov 25, 2015 · eba0141 · eba0141
2 parents 7c0baec + 59c1e33
commit eba0141
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/main/java/scmspain/karyon/restrouter/RestRouterHandler.java b/src/main/java/scmspain/karyon/restrouter/RestRouterHandler.java
@@ -99,7 +99,8 @@ public Observable<Void> handle(HttpServerRequest<ByteBuf> request,
 
       logError(throwable, request);
 
-      return Observable.empty();
+      response.getHeaders().set(HttpHeaders.CONTENT_TYPE, "text/plain");
+      return response.writeStringAndFlush("Internal server error");
     });
   }
 

diff --git a/src/test/java/scmspain/karyon/restrouter/RestRouterHandlerTest.java b/src/test/java/scmspain/karyon/restrouter/RestRouterHandlerTest.java
@@ -98,6 +98,9 @@ public void setUp() {
     given(serializerManager.getSupportedMediaTypes())
         .willReturn(ImmutableSet.of());
 
+    given(response.writeStringAndFlush(any()))
+        .willReturn(Observable.empty());
+
     ArgumentCaptor<Throwable> throwableArgumentCaptor = ArgumentCaptor.forClass(Throwable.class);
     given(errorHandler.handleError(eq(request), throwableArgumentCaptor.capture(), any()))
         .willAnswer(invocation -> Observable.error(throwableArgumentCaptor.getValue()));