Skip to content

Commit

Permalink
Merge pull request #731 from sul-dlss/remove-duplicate-logging-config
Browse files Browse the repository at this point in the history
  • Loading branch information
mjgiarlo authored Jan 16, 2025
2 parents a2e2dcf + 5fa5b6b commit 792c0ea
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 5 deletions.
5 changes: 0 additions & 5 deletions config/environments/production.rb
Original file line number Diff line number Diff line change
Expand Up @@ -41,11 +41,6 @@
# Force all access to the app over SSL, use Strict-Transport-Security, and use secure cookies.
config.force_ssl = true

# 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]

Expand Down
2 changes: 2 additions & 0 deletions config/initializers/lograge.rb
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,6 @@

Rails.application.configure do
config.lograge.enabled = Rails.env.production?
config.lograge.base_controller_class = ['ActionController::API', 'ActionController::Base']
config.lograge.ignore_actions = ['OkComputer::OkComputerController#show']
end

0 comments on commit 792c0ea

Please sign in to comment.