Fixes #7935 - Review HTTP/2 error handling (#8048)

Fixed HTTP/3 in similar way as HTTP/2. Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
jetty · May 23, 2022 · 513a2cb · 513a2cb
1 parent e4aff21
commit 513a2cb
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/...p3-server/src/main/java/org/eclipse/jetty/http3/server/internal/HttpChannelOverHTTP3.java b/...p3-server/src/main/java/org/eclipse/jetty/http3/server/internal/HttpChannelOverHTTP3.java
@@ -161,13 +161,11 @@ public Runnable onRequest(HeadersFrame frame)
         {
             if (LOG.isDebugEnabled())
                 LOG.debug("onRequest() failure", x);
-            onBadMessage(x);
-            return null;
+            return () -> onBadMessage(x);
         }
         catch (Throwable x)
         {
-            onBadMessage(new BadMessageException(HttpStatus.INTERNAL_SERVER_ERROR_500, null, x));
-            return null;
+            return () -> onBadMessage(new BadMessageException(HttpStatus.INTERNAL_SERVER_ERROR_500, null, x));
         }
     }