This is an automated email from the ASF dual-hosted git repository.
markt pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/9.0.x by this push:
new f01ef78305 Fix BZ 69471
f01ef78305 is described below
commit f01ef78305ceb8057ac594038084cc2fef684cbd
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 bd7f3de397..9a4736289a 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;
@@ -641,7 +642,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 5143d645aa..a5b18088f3 100644
--- a/webapps/docs/changelog.xml
+++ b/webapps/docs/changelog.xml
@@ -168,6 +168,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]
