Thanks for taking time out to answer my post.
On Jan 22, 2004, at 9:49 PM, Stas Bekman wrote:
stevan little wrote:All,
(I sent this originally to Matt Sergeant, and he suggested i post it here instead)
Apache::Reload a great module, there was however one thing that always drove me crazy. When a module reload failed, I found myself alot of times faced with the "Internal Server Error" page, and then i would have to sludge through the error log.
My proposed patch/addition basically wraps the require code in an eval block, and then if the require fails, it can catch the issue with [EMAIL PROTECTED] It also then replaces the current mod_perl content handler with one that will report the error to the browser. (It does this is a sorta convoluted way by first pushing a PerlFixupHandler which then re-assigns the PerlHandler. This is necessary because (for whatever reason) changes to the PerlHandler done before the URI translation phase will not stick.)
Thank you for the proposal, Stevan,
Personally I don't think this is a desired behavior to be in the core. Others may think differently.
Sorry, I have not been tracking mp2 dev all that closely (signed up today, and read some back messages about Apache::Reload this morning), so I didnt realize how Apache::Reload was being thought of for mp2. Given what you say about non-HTTP modules, my proposal doesn't make much sense then :)
Can't you just do:
CGI::Carp qw(fatalsToBrowser);
I have never tried this, I will test it out.
p.s. - I apologize for not sending this in the form of a patch, but I am new to the "contributing" end of open source software (after years of "using"). I will get better, I promise.
That would certainly be useful. Please read: http://perl.apache.org/contribute/cvs_howto.html#Sending_Patches and other docs under: http://perl.apache.org/contribute/
Thanks for the links, I am gonna review them right now.
- steve
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
