Jie Gao wrote:
Yeah, that was a very handy func. -Jie
If it's that simple to implement, and there is already one user reporting a desire to have it in mp2, I don't see a problem with it.
Apache::RequestUtil ?
On Wed, 22 Sep 2004, Stas Bekman wrote:
Date: Wed, 22 Sep 2004 22:13:27 -0400 From: Stas Bekman <[EMAIL PROTECTED]> To: mod_perl Dev <[EMAIL PROTECTED]> Subject: [mp2 api] log_reason
Over irc we have decided to dump log_reason. But now that I've looked at it, I'm not sure whether we should do that. If you call:
$r->log_reason("What's the reason?");
you get:
[Wed Sep 22 22:06:23 2004] [error] access to /My__Bug failed for 127.0.0.1, reason: What's the reason?
and there is no such equivalent API. in mp1 it was defined as:
#define mod_perl_log_reason(msg, file, r) \ aplog_error(APLOG_MARK, APLOG_ERR | APLOG_NOERRNO, r->server, \ "access to %s failed for %s, reason: %s", \ file, \ get_remote_host(r->connection, \ r->per_dir_config, REMOTE_NAME), \ msg)
I suppose we could just copy it over and then we don't need the compat.
What do you think?
-- __________________________________________________________________ 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]
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
-- -------------------------------------------------------------------------------- Philippe M. Chiasson m/gozer\@(apache|cpan|ectoplasm)\.org/ GPG KeyID : 88C3A5A5 http://gozer.ectoplasm.org/ F9BF E0C2 480E 7680 1AE5 3631 CB32 A107 88C3A5A5
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
