tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ethan Wallwork" <ethan.wallw...@net-linx.com>
Subject RE: BugRat Report #213 has been filed.
Date Mon, 30 Oct 2000 17:30:48 GMT

Has anyone looked at this?  It is a real problem for us, and if others agree it
should be changed I will get started on a fix.

--
Ethan

-----Original Message-----
From: BugRat Mail System [mailto:tomcat-bugs@cortexity.com]
Sent: Wednesday, October 04, 2000 2:13 PM
To: tomcat-bugs@cortexity.com
Subject: BugRat Report #213 has been filed.


Bug report #213 has just been filed.

You can view the report at the following URL:

   <http://znutar.cortexity.com:8888/BugRatViewer/ShowReport/213>

REPORT #213 Details.

Project: Tomcat
Category: Bug Report
SubCategory: New Bug Report
Class: swbug
State: received
Priority: medium
Severity: serious
Confidence: public
Environment:
   Release: 3.1, 3.2
   JVM Release: any
   Operating System: any
   OS Release: any
   Platform: any

Synopsis:
RequestDispatcher does not propagate errors

Description:
If a servlet uses a RequestDispatcher to forward or include a second servlet
(or JSP), and the second servlet throws and exception from it's service method,
that exception is not propagated to the first servlet.

This is a problem if you have a setup (like ours) where a single "master"
servlet includes a particular JSP depending on thing other than just the
request URI, and you want it to be able to handle the errors.  As it is, the
error gets handled by Tomcat (using default error page, or whatever error page
you've configured in the JSP or web.xml file) and the main servlet never knows
anything went wrong.

In Tomcat 3.2, the RequestDispatcher forward/include methods call the Handler
service method.  This service method never throws an exception.  In Tomcat 3.1
it calls the ServletWrapper handleRequest method, which also never throws and
exception.

The reason I think exceptions should be propagated is because of section 8.5 of
the servlet spec, which strongly implies it:

8.5 ERROR HANDLING
Only runtime exceptions and checked exceptions of type ServletException or
IOException should be propagated to the calling servlet if thrown by the target
of a request dispatcher.  All other exceptions should be wrapped as a
ServletException and the root cause of the exception set to the original
exception.




Mime
View raw message