Как я могу изменить настройки регистратора на основе переменной ENV в рубине с помощью регистратора?

Question

Я настроил следующее с помощью этого: Ruby - share logger instance among module/classes

EDIT: Основываясь на комментариях ниже, я обновил код, но все еще есть проблемы при входе.

module Logging 

    # in classes needing to be logged use 'include Logger' 
    require 'logger' 
    require 'remote_syslog_logger' 
    require_relative 'env' 

    class << self 
    def logger 

     return @logger if @logger 

     if ENV['ENVIRONMENT'] == 'production' #may need to change this 

     @logger ||= Logger.new($stdout) 

     @logger.level = Logger::WARN 
     @logger.datetime_format = "%N " 

     elsif ENV['ENVIRONMENT'] == 'development' 

     binding.pry 

     #$logger = Logger.new(STDOUT) 
     @logger = RemoteSyslogLogger.new('logs2.papertrailapp.com',39257) 
     @logger.level = Logger::DEBUG 
     @logger.datetime_format = "%N " 

     end 

    end 

    def logger=(logger) 
     @logger = logger 
    end 
    end 

    # Addition 
    def self.included(base) 
    class << base 
     def logger 
     Logging.logger 
     end 
    end 
    end 

    def logger 
    Logging.logger 
    end 


end 

Как войти:

class CronCheck 
    require_relative 'module_logger.rb' 
    include Logging 

    def self.run 
    begin 
     logger.debug "**** running cron_check_schedule.rb #{Time.now} #{Time.now.to_i}****" 
    rescue 
    end 
end 

Вопрос: Как я могу изменить модуль, так что этот тип логики может быть определен один раз, и все, что нужно сделать, это передать значение ENV для все классы и методы регистрируются таким же образом?

Я получаю сообщение об ошибке при logger.debug: неопределенный метод `отладки» для „% N“: String

Answer 1

Ваш код почти точно прав. Что вам нужно сделать, хотя, устанавливается Logging.logger, не $logger:

def self.instantiate_logger 
    logger = Logger.new(STDOUT) 
    logger.datetime_format = "%N " 

    if ENV['ENVIRONMENT'] == 'production' 
    logger.level = Logger::WARN 
    elsif ENV['ENVIRONMENT'] == 'development' 
    logger.level = Logger::DEBUG 
    end 

    Logging.logger = logger 
end 

Это не ясно из вашего вопроса, где этот метод живет, но он должен быть где-то можно назвать его во время инициализации вашего приложения.

Вы заметите, что я удалил код, связанный с Pry; поскольку этот код напрямую не связан с протоколированием, он должен, вероятно, перейти в другое место.

Я хотел бы сделать еще одно улучшение, хотя, который должен сделать его более декларативный, помещая уровни журнала в хэш шпонкой их соответствующими средами:

LOG_LEVELS = { 
    "production" => Logger::WARN, 
    "development" => Logger::DEBUG 
} 

def self.instantiate_logger 
    Logging.logger = Logger.new(STDOUT).tap do |logger| 
    logger.datetime_format = "%N " 
    logger.level = LOG_LEVELS[ENV['ENVIRONMENT']] || LOG_LEVELS["development"] 
    end 
end 

Answer 2

Предполагая, что первый фрагмент является module Logging вы предполагаете включить везде для регистрации (в противном случае глобальные переменные, как в второй фрагмент просто отлично работает из коробки,) все, что вам нужно, чтобы инициализировать регистратор в соответствии фрагмент # 2:

# this is the topmost method from your snippet #1, 
# updated to use env settings for logger tuning 
def logger 
    # @logger ||= Logger.new($stdout) 
    return @logger if @logger 

    @logger = case ENV['ENVIRONMENT'] 
      when 'production' 
       Logger.new(STDOUT).tap do |logger| 
       logger.level = Logger::WARN 
       logger.datetime_format = "%N " 
       end 
      else 
       ....... 
      end 
end 

Вот и все.