This is an automated email from the ASF dual-hosted git repository.
markt pushed a commit to branch 10.1.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/10.1.x by this push:
new f60ba37d9f Fix BZ 69471
f60ba37d9f is described below
commit f60ba37d9f09b1542d57d1948370dd7e257a80b1
Author: Mark Thomas <[email protected]>
AuthorDate: Fri Nov 22 18:42:27 2024 +0000
Fix BZ 69471
Log instances of CloseNowException caught by
ApplicationDispatcher.invoke() at debug level rather than error level as
they are very likely to have been caused by a client disconnection or
similar I/O issue.
https://bz.apache.org/bugzilla/show_bug.cgi?id=69471
---
java/org/apache/catalina/core/ApplicationDispatcher.java | 3 ++-
webapps/docs/changelog.xml | 6 ++++++
2 files changed, 8 insertions(+), 1 deletion(-)
diff --git a/java/org/apache/catalina/core/ApplicationDispatcher.java
b/java/org/apache/catalina/core/ApplicationDispatcher.java
index b662fe56c3..4fd7fc0585 100644
--- a/java/org/apache/catalina/core/ApplicationDispatcher.java
+++ b/java/org/apache/catalina/core/ApplicationDispatcher.java
@@ -46,6 +46,7 @@ import org.apache.catalina.connector.RequestFacade;
import org.apache.catalina.connector.Response;
import org.apache.catalina.connector.ResponseFacade;
import org.apache.coyote.BadRequestException;
+import org.apache.coyote.CloseNowException;
import org.apache.tomcat.util.ExceptionUtils;
import org.apache.tomcat.util.res.StringManager;
@@ -632,7 +633,7 @@ final class ApplicationDispatcher implements
AsyncDispatcher, RequestDispatcher
filterChain.doFilter(request, response);
}
// Servlet Service Method is called by the FilterChain
- } catch (BadRequestException e) {
+ } catch (BadRequestException | CloseNowException e) {
ioException = e;
} catch (IOException e) {
wrapper.getLogger().error(sm.getString("applicationDispatcher.serviceException",
wrapper.getName()), e);
diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
index 1c8c5986fb..1eeb731cc0 100644
--- a/webapps/docs/changelog.xml
+++ b/webapps/docs/changelog.xml
@@ -172,6 +172,12 @@
some headers, as per RFC 9110, section 9.3.2, that are present in a GET
request that are not present in the equivalent HEAD request. (markt)
</fix>
+ <fix>
+ <bug>69471</bug>: Log instances of <code>CloseNowException</code>
caught
+ by <code>ApplicationDispatcher.invoke()</code> at debug level rather
+ than error level as they are very likely to have been caused by a
client
+ disconnection or similar I/O issue. (markt)
+ </fix>
</changelog>
</subsection>
<subsection name="Coyote">
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
