Reduce log level for error that can occur in normal usage

apache · Dec 19, 2024 · 1afe414 · 1afe414
1 parent e380f5c
commit 1afe414
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/java/org/apache/tomcat/util/net/NioEndpoint.java b/java/org/apache/tomcat/util/net/NioEndpoint.java
@@ -684,7 +684,9 @@ public boolean events() {
                 SocketChannel sc = socketWrapper.getSocket().getIOChannel();
                 int interestOps = pe.getInterestOps();
                 if (sc == null) {
-                    log.warn(sm.getString("endpoint.nio.nullSocketChannel"));
+                    if (log.isDebugEnabled()) {
+                        log.debug(sm.getString("endpoint.nio.nullSocketChannel"));
+                    }
                     socketWrapper.close();
                 } else if (interestOps == OP_REGISTER) {
                     try {

diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
@@ -166,6 +166,11 @@
         Avoid a rare <code>NullPointerException</code> when recycling the
         <code>Http11InputBuffer</code>. (markt)
       </fix>
+      <fix>
+        Lower the log level to debug for logging an invalid socket channel when
+        processing poller events for the NIO Connector as this may occur in
+        normal usage. (markt)
+      </fix>
     </changelog>
   </subsection>
   <subsection name="Jasper">