You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@tiles.apache.org by "Clayton Rabenda (JIRA)" <ji...@apache.org> on 2010/10/06 21:22:31 UTC

[jira] Issue Comment Edited: (TILES-516) JSP exceptions are hidden by Tiles making debugging difficult

    [ https://issues.apache.org/jira/browse/TILES-516?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12918635#action_12918635 ] 

Clayton Rabenda edited comment on TILES-516 at 10/6/10 3:22 PM:
----------------------------------------------------------------

One other thing. There are ways to work around this, I have been able to simply catch the exceptions in the jsp myself using c:catch and display them directly in the html output so it's possilble that you could consider this a non-issue, but I would argue that hiding an exception is not preferable.

      was (Author: clayrab):
    One other thing. There are ways to work around this, I have been able to simply catch the exceptions in the jsp myself and display them directly in the html output so it's possilble that you could consider this a non-issue, but I would argue that hiding an exception is not preferable.
  
> JSP exceptions are hidden by Tiles making debugging difficult
> -------------------------------------------------------------
>
>                 Key: TILES-516
>                 URL: https://issues.apache.org/jira/browse/TILES-516
>             Project: Tiles
>          Issue Type: Bug
>            Reporter: Clayton Rabenda
>
> When using tiles:insertDefinition, any exceptions thrown by the jsp renderer are being hidden from the logs by TilesContainer.
> Based upon apache's logging best practices(http://commons.apache.org/logging/guide.html), I believe it would be best if exceptions caught in TilesContainer's render methods were logged before being rethrown.
> "External Boundaries - Expected Exceptions. This classification includes exceptions such as FileNotFoundException that cross API/SPI boundaries, and are exposed to the user of a component/toolkit. These are listed in the 'throws' clause of a method signature.
> Appropriate handling of these exceptions depends upon the type of code you are developing. API's for utility functions and tools should log these at the debug level, if they are caught at all by internal code.
> For higher level frameworks and middleware components, these exceptions should be caught immediatly prior to crossing the API/SPI interface back to user code-space, logged with full stack trace at info level, and rethrown. The assures that the log contains a record of the root cause for future analysis in the event that the exception is not caught and resolved as expected by the user's code. "

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.