Merge pull request #3888 from zendesk/grosser/ip

grosser · web-flow · commit ced287e41251 · 2021-01-11T13:44:22.000-08:00
remove duplicate client-ip from logs
diff --git a/config/logging.rb b/config/logging.rb
@@ -49,8 +49,7 @@
     request = event.payload[:headers].instance_variable_get(:@req)
     {
       params: params,
-      user_id: request.env['warden']&.user&.id,
-      client_ip: request.remote_ip
+      user_id: request.env['warden']&.user&.id
     }
   end
   config.lograge.formatter = Lograge::Formatters::Logstash.new

Original file line number	Diff line number	Diff line change
`@@ -49,8 +49,7 @@`
`49`	`49`	`request = event.payload[:headers].instance_variable_get(:@req)`
`50`	`50`	`{`
`51`	`51`	`params: params,`
`52`		`- user_id: request.env['warden']&.user&.id,`
`53`		`- client_ip: request.remote_ip`
	`52`	`+ user_id: request.env['warden']&.user&.id`
`54`	`53`	`}`
`55`	`54`	`end`
`56`	`55`	`config.lograge.formatter = Lograge::Formatters::Logstash.new`