summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndrew Newdigate <andrew@gitlab.com>2018-02-28 12:04:08 +0000
committerAndrew Newdigate <andrew@gitlab.com>2018-02-28 12:04:08 +0000
commitc869aa917c20b8289b3074431ec986c30b0b3834 (patch)
treec54c99885c7c181be44f908f176e629e35bd05de
parent0be4a77d0012613f960c4177f53101c46de2899c (diff)
downloadgitlab-ce-c869aa917c20b8289b3074431ec986c30b0b3834.tar.gz
Fix for open-ended parameter's in lograge causing elastic memory issues.
See https://gitlab.com/gitlab-com/infrastructure/issues/2888 See https://gitlab.com/gitlab-org/gitlab-ce/issues/43678 See https://gitlab.com/gitlab-com/migration/issues/163
-rw-r--r--config/initializers/lograge.rb5
1 files changed, 4 insertions, 1 deletions
diff --git a/config/initializers/lograge.rb b/config/initializers/lograge.rb
index 8560d24526f..f78f530e9a8 100644
--- a/config/initializers/lograge.rb
+++ b/config/initializers/lograge.rb
@@ -12,9 +12,12 @@ unless Sidekiq.server?
config.lograge.logger = ActiveSupport::Logger.new(filename)
# Add request parameters to log output
config.lograge.custom_options = lambda do |event|
+ params = event.payload[:params]
+ .except(*%w(controller action))
+ .each_pair.map { |k, v| { key: k, value: v } }
payload = {
time: event.time.utc.iso8601(3),
- params: event.payload[:params].except(*%w(controller action format)),
+ params: params,
remote_ip: event.payload[:remote_ip],
user_id: event.payload[:user_id],
username: event.payload[:username]