NullPointerException when there is an illegal character in the request (

helidon-io#5470)
klustria · Nov 18, 2022 · d888c52 · d888c52
1 parent f2d018d
commit d888c52
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/...ve/webserver/webserver/src/main/java/io/helidon/reactive/webserver/ForwardingHandler.java b/...ve/webserver/webserver/src/main/java/io/helidon/reactive/webserver/ForwardingHandler.java
@@ -340,6 +340,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 {
@@ -395,8 +398,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);