From 118adde47187fce3f362d0b5fff3d28406a82ff8 Mon Sep 17 00:00:00 2001 From: Christian Sutter Date: Thu, 28 Nov 2024 14:58:06 +0000 Subject: [PATCH] Fix logging for `UserEvents::Import` Turns out `govuk_app_config` overrides the nicely configured default Rails logger that supports tagging with its own basic logger. This wraps the Rails logger with `ActiveSupport::TaggedLogging` to ensure we can emit tags for the event type and date. --- app/services/discovery_engine/user_events/import.rb | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/app/services/discovery_engine/user_events/import.rb b/app/services/discovery_engine/user_events/import.rb index a64a51a..a2de75d 100644 --- a/app/services/discovery_engine/user_events/import.rb +++ b/app/services/discovery_engine/user_events/import.rb @@ -84,7 +84,9 @@ def partition_date end def logger - @logger ||= Rails.logger.tagged(self.class.name, event_type, date) + @logger ||= ActiveSupport::TaggedLogging + .new(Rails.logger) + .tagged(self.class.name, "event_type=#{event_type}", "date=#{date}") end end end