如何使用记录器根据 Ruby 中的 ENV 变量更改记录器设置

Question

我用这个设置了以下内容： Ruby - 在模块/类之间共享记录器实例

编辑：根据下面的评论，我更新了代码，但仍然有记录问题。

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"：字符串

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

就是这样。