benjaminoakes/airbrake — comment on commit b6e76017889da506040b409839a1e3dce0729341
File: lib/airbrake/rails/controller_methods.rb line 43
By shime:

yes, ben. this code is relevant to rails controller methods, so I don't think it makes difference if you use params instead of hash but it's cleaner in the method context.

is this variable assignment really necessary? can you please just use a one liner like `ActionDispatch::Http::ParameterFilter.new(::Rails.application.config.filter_parameters).filter hash`.

This is also not safe from exceptions, so please add the `rescue` statement. I would prefer if you added it after the `end` statement so it's more DRY. your thoughts on this?

thank you for contributing once again