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.
Can't you just do:
CGI::Carp qw(fatalsToBrowser);
re: mp2
In mp2 Apache::Reload is also used to reload modules which has nothing to do with HTTP. So it can be run as PerlPreConnection handler, when there is no HTTP client to send the information to.
Also in mp2 you can't send any response (besides error codes) before the PerlResponseHandler, so it just won't work.
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/
Looking forward for your other contributions.
__________________________________________________________________ Stas Bekman JAm_pH ------> Just Another mod_perl Hacker http://stason.org/ mod_perl Guide ---> http://perl.apache.org mailto:[EMAIL PROTECTED] http://use.perl.org http://apacheweek.com http://modperlbook.org http://apache.org http://ticketmaster.com
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
