[Yanel-dev] Generic exception handler

[Cedric Staub]

On Thu, Jan 27, 2011 at 12:07:21AM +0100, Michael Wechner wrote:
> On 1/25/11 2:38 PM, Cedric Staub wrote:
> > On Tue, Jan 25, 2011 at 09:35:34AM +0000, Michael Wechner wrote:
> >> On 1/24/11 10:03 AM, Cedric Staub wrote:
> >>> Hello there!
> >>>
> >>> I wrote a resource called generic-exception-handler, which is called
> >>> whenever an exception is thrown while a request is being processed.
> >>> Yanel had this functionality built-in until now, but that made it
> >>> impossible to overwrite the exception-handler with a custom version.
> >>>
> >>> With this extension it's possible to write your own custom exception
> >>> handler and overwrite the generic exception handler's global resource
> >>> config. That way you can get rid of the Yanel-specific error pages and
> >>> install you own ones ;-).
> >>
> >> thanks very much :-) Can you please add it to Bugzilla and I will try to
> >> review it
> >> shortly
> > See bug #8115.
> 
> it seems like this patch contains more files than the original post ;-)

Yeah there was some stuff missing.

> >>> Note: The exception and stack trace are logged but not displayed in the
> >>> browser. I don't feel the user needs to know the details of the inner
> >>> workings of the application. The dev or sysadmin can just look at the
> >>> logs anyway, and I feel stack traces are confusing to users.
> >>>
> >>> But what to others think? Stack trace: yes or no?
> >> I understand that the stacktrace to the user can be odd, but on the
> >> other hand some
> >> users will copy/paste it and send it to some developer, which can be
> >> useful, because
> >> it might be a "one time" error which can be hard to find otherwise
> >> within huge log-files.
> >>
> >> Hence I would suggest that the  stacktrace is "extensible" (e.g. via
> >> CSS/Javascript)
> > Ok, but do you have any ideas an how I could pass the Exception object
> > to the resource?
> 
> not yet, but will tackle this shortly after I will have reviewed the 
> patch from above.

Ok cool.

Cheers