[TAP5-2133] DefaultRequestExceptionHandler swallows exceptions

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 5.4
• Fix Version/s: 5.4
• Component/s: tapestry-core
• Assignee: Howard Lewis Ship
• Reporter: Jochen Kemnade
• Created: 25/Jun/13 09:24
• Updated: 23/Jun/14 16:48
• Resolved: 23/Jun/14 16:22

Description

If an exception is thrown from the try block in org.apache.tapestry5.internal.services.DefaultRequestExceptionHandler.handleRequestException(Throwable), it is swallowed without any notification. In my case, the problem was a LinkCreationListener's throwing a NPE in a redirected request originating from a previous exception. This was very hard to debug because the NPE didn't show up in any stack trace or log.
I suggest that either the handler fail completely if the try block throws an exception or at least log it.