[seam-issues] [JBoss JIRA] Commented: (SEAMSERVLET-19) Catch integration swallows unhandled runtime exceptions
Dan Allen (JIRA)
jira-events at lists.jboss.org
Wed Dec 15 11:34:17 EST 2010
[ https://issues.jboss.org/browse/SEAMSERVLET-19?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12570441#comment-12570441 ]
Dan Allen commented on SEAMSERVLET-19:
--------------------------------------
Doh! I was focusing on the dang checked exceptions that I forgot about the non-checked ones ;) Will definitely fix.
> Catch integration swallows unhandled runtime exceptions
> -------------------------------------------------------
>
> Key: SEAMSERVLET-19
> URL: https://issues.jboss.org/browse/SEAMSERVLET-19
> Project: Seam Servlet
> Issue Type: Bug
> Components: Catch integration
> Affects Versions: 3.0.0.Alpha2
> Reporter: Jozef Hartinger
> Priority: Critical
> Fix For: 3.0.0.Alpha3
>
>
> The spec says:
> If the servlet that is the target of a request dispatcher throws a runtime exception or
> a checked exception of type ServletException or IOException, it should be
> propagated to the calling servlet. All other exceptions should be wrapped as
> ServletExceptions and the root cause of the exception set to the original exception,
> as it should not be propagated.
> CatchExceptionFilter contains this code:
> if (!catchEvent.isHandled())
> {
> if (e instanceof ServletException)
> {
> throw (ServletException) e;
> }
> else if (e instanceof IOException)
> {
> throw (IOException) e;
> }
> }
> which takes care of ServletExceptions and IOExceptions, however, runtime exceptions are swallowed.
--
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
More information about the seam-issues
mailing list