[Catalyst] C::Log::Log4perl looses track

Adam Jacob adam at stalecoffee.org
Fri Mar 9 20:17:18 GMT 2007


This has been pushed to CPAN, and should resolve this issue for you.

Adam

On Feb 20, 2007, at 2:33 PM, Adam Jacob wrote:

> There is actually a patch for this in the pipeline.  I'll attach it  
> to this message; let me know if it fixes this for you.  I haven't  
> had time to give it a proper testing, sadly, so there hasn't been a  
> new release of C::L::Log4perl because of it.
>
> I'm getting married on Saturday (yay!) and so probably won't be of  
> much help getting a new CPAN release pushed.  Several folks on  
> #catalyst and #catalyst-dev have PAUSE permissions to do so,  
> though, if we need to.
>
> Otherwise, it's on my list of things to do once things settle down  
> a bit for me.
>
> It is definitely a bug.
>
> Adam
>
> On Feb 20, 2007, at 9:02 AM, Cédric Bouvier wrote:
>
>> Hello,
>>
>> I'm trying to use Catalyst::Log::Log4perl and I would like it to  
>> print
>> the name of the method it was called from and the line number,  
>> instead
>> of the default "INFO - message".
>>
>> Unfortunately, it seem to always consider it was called from
>> Catalyst::handle_request, at line 1479. This line calls
>> C::L::Log4perl::_flush, which indeed call log4perl's appropriate  
>> method.
>>
>> As far as I understood, C::L::L only pushes log messages to an  
>> arrayref,
>> and sends them to log4perl only when _flush() is invoked, and that
>> happens *after* the request has been handled. The messages are  
>> stored in
>> the arrayref together with their caller's package, and depth (how  
>> deep
>> in the call stack they are), but when they are finally sent to  
>> Log4perl,
>> most of the context is gone.
>>
>> My questions:
>> - am I missing something obvious?
>> - is this a bug? or is there a good reason?
>> - out of sheer curiosity, why not log at once?
>> - are the authors on the list and willing to turn this postponed  
>> logging
>>   into a configurable feature (I can do it myself and provide a  
>> patch,
>>   if need be)
>>
>> TIA
>>
>> -- 
>> C é d r i c   B o u v i e r
>> _______________________________________________
>> List: Catalyst at lists.rawmode.org
>> Listinfo: http://lists.rawmode.org/mailman/listinfo/catalyst
>> Searchable archive: http://www.mail-archive.com/ 
>> catalyst at lists.rawmode.org/
>> Dev site: http://dev.catalyst.perl.org/
>




More information about the Catalyst mailing list