NullPointerException when there is an illegal character in the request (

#5470)
helidon-io · Nov 17, 2022 · 643ee02 · 643ee02
1 parent f44f7da
commit 643ee02
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/webserver/webserver/src/main/java/io/helidon/webserver/ForwardingHandler.java b/webserver/webserver/src/main/java/io/helidon/webserver/ForwardingHandler.java
@@ -351,6 +351,9 @@ private boolean channelReadHttpRequest(ChannelHandlerContext ctx, Context reques
         // New request ID
         long requestId = REQUEST_ID_GENERATOR.incrementAndGet();
 
+
+        requestEntityAnalyzed = new CompletableFuture<>();
+
         // If a problem with the request URI, return 400 response
         BareRequestImpl bareRequest;
         try {
@@ -405,8 +408,6 @@ private boolean channelReadHttpRequest(ChannelHandlerContext ctx, Context reques
             prevRequestFuture = null;
         }
 
-        requestEntityAnalyzed = new CompletableFuture<>();
-
         //If the keep alive is not set, we know we will be closing the connection
         if (!HttpUtil.isKeepAlive(requestContext.request())) {
             this.requestEntityAnalyzed.complete(ChannelFutureListener.CLOSE);