On Mon, Feb 12, 2001 at 10:50:55AM -0800, Jon Stevens wrote:
> on 2/12/01 4:59 AM, "Sean Legassick" <[EMAIL PROTECTED]> wrote:
> 
> > A piece of functionality I really want is the ability to email errors to
> > an administrative email address. Previously I had thought that making
> > the exception handling in Turbine.handleException pluggable was the way
> > to go, but looking at the (wonderful) new LoggingService I think the
> > right way to go is to write an EmailLogger class ...
> > 
> > Can anyone forsee any problems with this, or shall I go ahead and write
> > the class?
> > 
> > Sean
> 
> Well, there are two levels of errors in Turbine. There are exceptions that
> are thrown and there are Log.error() messages. If you and an email each and
> every time a Log.error() is hit, then using a EmailLogger would be good. If
> you want an email sent when an exception is thrown up to the top level
> Turbine.java class, then a *ErrorScreen is best.

Oh yeah, I hadn't thought of putting the functionality in the
ErrorScreen. Good idea.

I quite like the idea of emailing Log.error though, because sometimes
exceptions don't get propogated all the way up (like an exception thrown
by a method invoked during template merging).

So I think I'll go ahead and do an EmailLogger, but maybe look at adding
email functionality to error screens as well. In fact I feel a general
purpose exception-mailing service coming on...

Sean
 
-- 
Sean Legassick
[EMAIL PROTECTED]
      Hombre soy, nada humano me puede ser ajeno  
      
      


------------------------------------------------------------
To subscribe:        [EMAIL PROTECTED]
To unsubscribe:      [EMAIL PROTECTED]
Search: <http://www.mail-archive.com/turbine%40list.working-dogs.com/>
Problems?:           [EMAIL PROTECTED]

  • ... Sean Legassick
    • ... Κώστας Στεργίου
      • ... Jon Stevens
        • ... Daniel Rall
          • ... Κώστας Στεργίου
            • ... Daniel Rall
    • ... Jon Stevens
      • ... Sean Legassick
        • ... Daniel Rall
    • ... Daniel Rall
      • ... Sean Legassick
        • ... Sean Legassick
          • ... Daniel Rall
        • ... Daniel Rall

Reply via email to