log4perl-devel

[log4perl-devel] "Filter" attribute to standard appenders

[Alexey M. <al...@w-...>]

Hello,

I have just spent twenty minutes trying to understand why my custom
filter did not work.

I have the following definition:

        log4perl.appender.DefaultAppender = Log::Log4perl::Appender::Screen
        # WARNING: here "Filter" should be in title case, not in lowercase
        log4perl.appender.DefaultAppender.Filter = RelevanceFilter

Unfortunately, first time I've typed the "Filter" in lowercase, and it
did not attach the filter to screen appender.  No error message, no
warning.  Very counter-intuitive, especially because filter definition
itself uses lower-case:

       log4perl.filter.RelevanceFilter = PR::Logger::RelevanceFilter

Especially because other appender's attributes are lowercase:

       log4perl.appender.Screen.Filter = MyFilter
       log4perl.appender.Screen.layout = Log::Log4perl::Layout::SimpleLayout


Could you somehow make attributes case-insensitive?



Otherwise, Log::Log4perl is extremely useful tool.


Thanks,

--alexm

Re: [log4perl-devel] "Filter" attribute to standard appenders

[Mike S. <log...@pe...>]

On Wed, 16 Jul 2003, Alexey Mahotkin wrote:

>         log4perl.appender.DefaultAppender = Log::Log4perl::Appender::Screen
>         # WARNING: here "Filter" should be in title case, not in lowercase
>         log4perl.appender.DefaultAppender.Filter = RelevanceFilter
>
> Unfortunately, first time I've typed the "Filter" in lowercase, and it
> did not attach the filter to screen appender.  No error message, no
> warning.  Very counter-intuitive, especially because filter definition
> itself uses lower-case:
>
>        log4perl.filter.RelevanceFilter = PR::Logger::RelevanceFilter

I agree that it's somewhat confusing -- reason for the strange uppercasing
convention in "log4perl.appender.DefaultAppender.Filter" is that "Filter"
is a special keyword, not a specific appender property, like "file"
or "mode" for a Log::Log4perl::Appender::File appender.

It's probably too harsh to issue a warning in this case, because
theoretically, you could have a specific appender property called
"filter" (although that's not recommended).

It's a compromise.

-- Mike

Mike Schilli
log...@pe...
http://perlmeister.com
http://log4perl.sourceforge.net