Merge branch 'sentry-filter' into 'master'

Prevent Rails filtered parameters from leaking to Sentry. As described in their Docs: https://docs.getsentry.com/on-premise/clients/ruby/integrations/rails/ See merge request !3974

Merge branch 'sentry-filter' into 'master'
ccd3f707 · Robert Speicher · 85d64032 · 519a791e · ccd3f707
Commit ccd3f707 authored Apr 29, 2016 by Robert Speicher
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 0 deletions

sentry.rb config/initializers/sentry.rb +3 -0

No files found.
--- a/config/initializers/sentry.rb
+++ b/config/initializers/sentry.rb
@@ -15,6 +15,9 @@ if Rails.env.production?
    Raven.configure do |config|
      config.dsn = current_application_settings.sentry_dsn
      config.release = Gitlab::REVISION
+      
+      # Sanitize fields based on those sanitized from Rails.
+      config.sanitize_fields = Rails.application.config.filter_parameters.map(&:to_s)
    end
  end
 end