From 2d2217b677710e65c1808f26fa8747eb22867ce6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jonne=20Ha=C3=9F?= Date: Sat, 20 Sep 2014 03:01:43 +0200 Subject: [PATCH] Monkey patch Sidekiq logger Sidekiq 3.0 initialized the logger early before we had a chance to set the logfile. Then we set a new logfile causing the old logger to be closed. Since the default logger points to stdout, it got closed causing all kinds of issues. --- config/initializers/sidekiq.rb | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/config/initializers/sidekiq.rb b/config/initializers/sidekiq.rb index a1ff43592..c0c6da556 100644 --- a/config/initializers/sidekiq.rb +++ b/config/initializers/sidekiq.rb @@ -11,6 +11,9 @@ if AppConfig.environment.single_process_mode? && Rails.env != "test" require 'sidekiq/testing/inline' end +def (Sidekiq::Logging).logger + defined?(@logger) ? @logger : (AppConfig.heroku? ? initialize_logger : initialize_logger(AppConfig.sidekiq_log)) +end Sidekiq.configure_server do |config| config.redis = AppConfig.get_redis_options @@ -36,8 +39,6 @@ Sidekiq.configure_server do |config| chain.add SidekiqMiddlewares::CleanAndShortBacktraces end - Sidekiq::Logging.initialize_logger AppConfig.sidekiq_log unless AppConfig.heroku? - # Set connection pool on Heroku database_url = ENV['DATABASE_URL'] if(database_url)