add request id to operational logs; use json format

app/lib/audit_logger.rb

@@ -1,6 +1,6 @@
 class AuditLogger < ActiveSupport::Logger
   def initialize
     super(Config[:audit_log_file])
-    self.formatter = AuditLogFormatter.new
+    self.formatter = JsonLogFormatter.new
   end
 end

app/lib/audit_log_formatter.rb → app/lib/json_log_formatter.rb

@@ -1,4 +1,4 @@
-class AuditLogFormatter < ActiveSupport::Logger::SimpleFormatter
+class JsonLogFormatter < ActiveSupport::Logger::SimpleFormatter
   def call(severity, timestamp, _progname, message)
     # request_id is unique to the life of the api request
     request_id = Thread.current[:request_id]

config/environments/production.rb

@@ -47,14 +47,6 @@
   # Skip http-to-https redirect for the default health check endpoint.
   # config.ssl_options = { redirect: { exclude: ->(request) { request.path == "/up" } } }
 
-  # Log to STDOUT by default
-  config.logger = ActiveSupport::Logger.new(STDOUT)
-    .tap  { |logger| logger.formatter = ::Logger::Formatter.new }
-    .then { |logger| ActiveSupport::TaggedLogging.new(logger) }
-
-  # Prepend all log lines with the following tags.
-  config.log_tags = [ :request_id ]
-
   # "info" includes generic and useful information about system operation, but avoids logging too much
   # information to avoid inadvertent exposure of personally identifiable information (PII). If you
   # want to log everything, set the level to "debug".

config/initializers/logging.rb

@@ -0,0 +1,4 @@
+Rails.application.config.to_prepare do
+  Rails.logger = ActiveSupport::Logger.new(STDOUT)
+  Rails.logger.formatter = JsonLogFormatter.new
+end