add simple logging about HttpResponseStatus.TOO_MANY_REQUESTS and Htt…

…pResponseStatus.INTERNAL_SERVER_ERROR (#58)
teragrep · May 6, 2024 · c9de197 · c9de197
1 parent 925d21b
commit c9de197
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/main/java/com/teragrep/lsh_01/MessageProcessor.java b/src/main/java/com/teragrep/lsh_01/MessageProcessor.java
@@ -72,6 +72,7 @@ public class MessageProcessor implements RejectableRunnable {
     public void onRejection() {
         try {
             final FullHttpResponse response = generateFailedResponse(HttpResponseStatus.TOO_MANY_REQUESTS);
+            LOGGER.warn("Too many requests, returning code <{}>", response.status().code());
             ctx.writeAndFlush(response);
         }
         finally {
@@ -140,7 +141,9 @@ private FullHttpResponse processMessage(Subject subject) {
             return generateResponse(messageHandler.responseHeaders());
         }
         else {
-            return generateFailedResponse(HttpResponseStatus.INTERNAL_SERVER_ERROR);
+            FullHttpResponse response = generateFailedResponse(HttpResponseStatus.INTERNAL_SERVER_ERROR);
+            LOGGER.warn("Processing failed, returning code <{}>", response.status().code());
+            return response;
         }
     }