Refactor Logtail logger configuration in application.rb to conditionally set logger based on environment variables

Author: Shpigford

config/application.rb

@@ -26,5 +26,9 @@ class Application < Rails::Application
     # config.time_zone = "Central Time (US & Canada)"
     # config.eager_load_paths << Rails.root.join("extras")
     config.active_support.to_time_preserves_timezone = :zone
+
+    if ENV["LOGTAIL_API_KEY"].present? && ENV["LOGTAIL_INGESTING_HOST"].present?
+      config.logger = Logtail::Logger.create_default_logger(ENV["LOGTAIL_API_KEY"], ingesting_host: ENV["LOGTAIL_INGESTING_HOST"])
+    end
   end
 end

config/environments/production.rb

@@ -45,18 +45,6 @@
   # Force all access to the app over SSL, use Strict-Transport-Security, and use secure cookies.
   config.force_ssl = true
 
-  # Log to Logtail if API key is present, otherwise log to STDOUT
-  config.logger = if ENV["LOGTAIL_API_KEY"].present?
-    Logtail::Logger.create_default_logger(
-      ENV["LOGTAIL_API_KEY"],
-      ingesting_host: ENV["LOGTAIL_INGESTING_HOST"]
-    )
-  else
-    ActiveSupport::Logger.new(STDOUT)
-      .tap  { |logger| logger.formatter = ::Logger::Formatter.new }
-      .then { |logger| ActiveSupport::TaggedLogging.new(logger) }
-  end
-
   # Prepend all log lines with the following tags.
   config.log_tags = [ :request_id ]