Re: [Mason] Sub-classing request to change error-handling

2007-05-15 Thread Ian Burrell
On 5/15/07, Ian Burrell <[EMAIL PROTECTED]> wrote: > On 5/15/07, Jonathan Swartz <[EMAIL PROTECTED]> wrote: > > Sorry. What I mean is, change alter_superclass to just contain the > > class you are overriding (I assume that is > > HTML::Mason::Request::CGI). The code may be from the docs, but it > >

Re: [Mason] Sub-classing request to change error-handling

2007-05-15 Thread Ian Burrell
On 5/15/07, Jonathan Swartz <[EMAIL PROTECTED]> wrote: > Sorry. What I mean is, change alter_superclass to just contain the > class you are overriding (I assume that is > HTML::Mason::Request::CGI). The code may be from the docs, but it > seems a little suspect to me. For example, one could have >

Re: [Mason] Sub-classing request to change error-handling

2007-05-15 Thread Jonathan Swartz
Sorry. What I mean is, change alter_superclass to just contain the class you are overriding (I assume that is HTML::Mason::Request::CGI). The code may be from the docs, but it seems a little suspect to me. For example, one could have H::M::ApacheHandler loaded even if one was using H::M::CG

Re: [Mason] Sub-classing request to change error-handling

2007-05-15 Thread Ian Burrell
On 5/15/07, Jonathan Swartz <[EMAIL PROTECTED]> wrote: > Could you simplify the code you were using and take out the if() > statement, or at least log which branch is being taken? > What if statement? I got the error message even when all the request subclass contains is the new() method from the

Re: [Mason] Sub-classing request to change error-handling

2007-05-15 Thread Jonathan Swartz
Could you simplify the code you were using and take out the if() statement, or at least log which branch is being taken? On May 15, 2007, at 12:45 PM, Ian Burrell wrote: > On 5/15/07, Jeremy Blain <[EMAIL PROTECTED]> wrote: >> You could do this in the handler under mod perl. >> > > We aren't us

Re: [Mason] Sub-classing request to change error-handling

2007-05-15 Thread Ian Burrell
On 5/15/07, Jeremy Blain <[EMAIL PROTECTED]> wrote: > You could do this in the handler under mod perl. > We aren't using mod_perl. We are using CGIHandler and SpeedyCGI. Unfortunately, wrapping handle_cgi_object doesn't work since the error code needs access to the mason request and cgi_request F

Re: [Mason] Sub-classing request to change error-handling

2007-05-15 Thread Jeremy Blain
You could do this in the handler under mod perl. sub handler { my ($r) = @_; my $result = eval { $ah->handle_request($r); }; if (my $err = $@) { # log it to error log $r->log_error($err->as_line); ... email developers required info ... return 500; } retur