Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

working around that filter_parameters is protected now

Signed-off-by: Michael Koziarski <michael@koziarski.com>
  • Loading branch information...
commit 019b0de35a328c78204adc0fe93e942f011e0002 1 parent dfd2129
Trevor Turk authored NZKoz committed
Showing with 2 additions and 2 deletions.
  1. +2 −2 lib/exception_notifier_helper.rb
View
4 lib/exception_notifier_helper.rb
@@ -67,12 +67,12 @@ def exclude_raw_post_parameters?
end
def filter_sensitive_post_data_parameters(parameters)
- exclude_raw_post_parameters? ? @controller.filter_parameters(parameters) : parameters
+ exclude_raw_post_parameters? ? @controller.send!(:filter_parameters, parameters) : parameters
end
def filter_sensitive_post_data_from_env(env_key, env_value)
return env_value unless exclude_raw_post_parameters?
return PARAM_FILTER_REPLACEMENT if (env_key =~ /RAW_POST_DATA/i)
- return @controller.filter_parameters({env_key => env_value}).values[0]
+ return @controller.send!(:filter_parameters, {env_key => env_value}).values[0]
end
end

5 comments on commit 019b0de

@trevorturk

Thanks for getting this in. I’m going to kill my unnecessary fork now :)

@duff

For some reason, send! is coming up as an undefined method for me. Am I doing something wrong? Perhaps I need a different Rails revision? I’m about 2 week behind edge. If I call just send without the bang, it seems to work.

Thanks!

@trevorturk

http://rails.lighthouseapp.com/projects/8995/tickets/35-objectsend-removal-from-edge-rails-breaks-exception_notification

@duff

Yay! thanks.

@sxross

Thanks for this.

Please sign in to comment.
Something went wrong with that request. Please try again.