Only log controller parameters when configured to.

README.md

@@ -59,6 +59,18 @@ In your Gemfile:
     # This line is optional if you do not want to suppress app logs in your <environment>.log
     config.logstasher.suppress_app_log = false
 
+## Logging params hash
+
+Logstasher can be configured to log the contents of the params hash.  When enabled, the contents of the params hash (minus the ActionController internal params)
+will be added to the log as a deep hash.  This can cause conflicts within the Elasticsearch mappings though, so should be enabled with care.  Conflicts will occur
+if different actions (or even different applications logging to the same Elasticsearch cluster) use the same params key, but with a different data type (e.g. a
+string vs. a hash).  This can lead to lost log entries.  Enabling this can also significantly increase the size of the Elasticsearch indexes.
+
+To enable this, add the following to your `<environment>.rb`
+
+    # Enable logging of controller params
+    config.logstasher.log_controller_parameters = true
+
 ## Adding custom fields to the log
 
 Since some fields are very specific to your application for e.g. *user_name*, so it is left upto you, to add them. Here's how to add those fields to the logs:

lib/logstasher.rb

@@ -6,7 +6,7 @@
 
 module LogStasher
   # Logger for the logstash logs
-  mattr_accessor :logger, :enabled
+  mattr_accessor :logger, :enabled, :log_controller_parameters
 
   def self.remove_existing_log_subscriptions
     ActiveSupport::LogSubscriber.log_subscribers.each do |subscriber|
@@ -33,8 +33,11 @@ def self.unsubscribe(component, subscriber)
   def self.add_default_fields_to_payload(payload, request)
     payload[:ip] = request.remote_ip
     payload[:route] = "#{request.params[:controller]}##{request.params[:action]}"
-    payload[:parameters] = payload[:params].except(*ActionController::LogSubscriber::INTERNAL_PARAMS)
-    self.custom_fields += [:ip, :route, :parameters]
+    self.custom_fields += [:ip, :route]
+    if self.log_controller_parameters
+      payload[:parameters] = payload[:params].except(*ActionController::LogSubscriber::INTERNAL_PARAMS)
+      self.custom_fields += [:parameters]
+    end
   end
 
   def self.add_custom_fields(&block)
@@ -52,6 +55,7 @@ def self.setup(app)
     self.logger = app.config.logstasher.logger || Logger.new("#{Rails.root}/log/logstash_#{Rails.env}.log")
     self.logger.level = app.config.logstasher.log_level || Logger::WARN
     self.enabled = true
+    self.log_controller_parameters = !! app.config.logstasher.log_controller_parameters
   end
 
   def self.suppress_app_logs(app)

spec/lib/logstasher/log_subscriber_spec.rb

@@ -14,8 +14,12 @@ def log_output.json
   }
   before do
     LogStasher.logger = logger
+    LogStasher.log_controller_parameters = true
     LogStasher.custom_fields = []
   end
+  after do
+    LogStasher.log_controller_parameters = false
+  end
 
   let(:subscriber) {LogStasher::RequestLogSubscriber.new}
   let(:event) {

spec/lib/logstasher_spec.rb

@@ -38,15 +38,24 @@
     let(:request) { double(:params => params, :remote_ip => '10.0.0.1')}
     after do
       LogStasher.custom_fields = []
+      LogStasher.log_controller_parameters = false
     end
     it 'appends default parameters to payload' do
+      LogStasher.log_controller_parameters = true
       LogStasher.custom_fields = []
       LogStasher.add_default_fields_to_payload(payload, request)
       payload[:ip].should == '10.0.0.1'
       payload[:route].should == 'test#action'
       payload[:parameters].should == {'a' => '1', 'b' => 2}
       LogStasher.custom_fields.should == [:ip, :route, :parameters]
     end
+
+    it 'does not include parameters when not configured to' do
+      LogStasher.custom_fields = []
+      LogStasher.add_default_fields_to_payload(payload, request)
+      payload.should_not have_key(:parameters)
+      LogStasher.custom_fields.should == [:ip, :route]
+    end
   end
 
   describe '.append_custom_params' do
@@ -59,7 +68,7 @@
 
   describe '.setup' do
     let(:logger) { double }
-    let(:logstasher_config) { double(:logger => logger,:log_level => 'warn') }
+    let(:logstasher_config) { double(:logger => logger,:log_level => 'warn',:log_controller_parameters => nil) }
     let(:config) { double(:logstasher => logstasher_config) }
     let(:app) { double(:config => config) }
     before do
@@ -74,6 +83,7 @@
       LogStasher.setup(app)
       LogStasher.enabled.should be_true
       LogStasher.custom_fields.should == []
+      LogStasher.log_controller_parameters.should == false
     end
   end