Merge pull request #3272 from mxsm/eventmesh-3271

[ISSUE #3271]Fix handle http message throw IllegalReferenceCountException
apache · Feb 27, 2023 · 3e9b1ee · 3e9b1ee
2 parents 7955f36 + 3a36275
commit 3e9b1ee
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/eventmesh-runtime/src/main/java/org/apache/eventmesh/runtime/boot/AbstractHTTPServer.java b/eventmesh-runtime/src/main/java/org/apache/eventmesh/runtime/boot/AbstractHTTPServer.java
@@ -95,7 +95,6 @@
 import io.netty.handler.ssl.SslHandler;
 import io.netty.handler.timeout.IdleState;
 import io.netty.handler.timeout.IdleStateEvent;
-import io.netty.util.ReferenceCountUtil;
 import io.opentelemetry.api.trace.Span;
 import io.opentelemetry.semconv.trace.attributes.SemanticAttributes;
 
@@ -444,9 +443,7 @@ protected void channelRead0(ChannelHandlerContext ctx, HttpRequest httpRequest)
                 }
 
             } catch (Exception ex) {
-                log.error("AbrstractHTTPServer.HTTPHandler.channelRead error", ex);
-            } finally {
-                ReferenceCountUtil.release(httpRequest);
+                log.error("execute AbstractHTTPServer.HTTPHandler.channelRead0 error", ex);
             }
         }