Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Log4r::Logger.add incompatible with Ruby's Logger.add? #30

Open
ghost opened this issue Mar 31, 2014 · 1 comment
Open

Log4r::Logger.add incompatible with Ruby's Logger.add? #30

ghost opened this issue Mar 31, 2014 · 1 comment

Comments

@ghost
Copy link

ghost commented Mar 31, 2014

I have log4r working successfully in Rails 3.2.17 app running on Windows Server 2008 R2. However, when I tried starting a DelayedJob worker via rake I found it raised an error.

Basically, DelayedJob is calling logger.add (see delayed_job/lib/delayed/worker.rb line 248):

logger.add level, "#{Time.now.strftime('%FT%T%z')}: #{text}" if logger

This is valid for Ruby's Logger class (see Logger.add), however for log4r it tries to add an outputter in Log4r::Logger.add (see logger.rb line 119)

Is this expected behavior? Am I missing something?

@lukeasrodgers
Copy link

FWIW this appears to no longer be an issue on delayedjob master, which uses .send(:level... instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant