This project is archived and is in readonly mode.

#3763 ✓resolved
Prem Sichanugrist (sikachu)

[PATCH] Move filter_parameter_logging logic into ActionDispatch::ParametersFilter

Reported by Prem Sichanugrist (sikachu) | January 21st, 2010 @ 04:50 AM | in 3.0.2

I was going to implement a feature to filter request_uri, and found out that filter_parameter_logging is defined in controller instead. So I write this patch to move the logic into ActionDispatch:: ParametersFilter, which is included in ActionDispatch::Request

I also saw this commit implements filter_env, so I'm implementing it in ActionDispatch::ParametersFilter also.

I've separate the patch into two parts: one is the ActionDispatch::ParametersFilter itself, and the other one is deprecation of filter_parameter_logging in controller. I think the configuration should be moved to Rails.configuration instead.

Comments and changes to this ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

<h2 style="font-size: 14px">Tickets have moved to Github</h2>

The new ticket tracker is available at <a href=""></a>