Skip to content

Commit e5fbdde

Browse files
committed
filter params correctly from error log
1 parent 60284d8 commit e5fbdde

1 file changed

Lines changed: 1 addition & 14 deletions

File tree

app/models/error_log.rb

Lines changed: 1 addition & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ def self.report!(exception, controller, request, user)
2929
date: DateTime.now,
3030
guid: SecureRandom.uuid,
3131
user_id: user && user.id,
32-
request: filter_sensitive_post_data_parameters(controller, request.parameters).inspect,
32+
parameters: request && request.filtered_parameters.to_json,
3333
action: controller.action_name,
3434
controller: controller.controller_name,
3535
backtrace: sanitize_backtrace(exception.backtrace).join("\n"),
@@ -92,17 +92,4 @@ def self.sanitize_backtrace(trace)
9292
trace.map { |line| Pathname.new(line.gsub(re, "[RAILS_ROOT]")).cleanpath.to_s }
9393
end
9494

95-
def self.exclude_raw_post_parameters?(controller)
96-
controller && controller.respond_to?(:filter_parameters)
97-
end
98-
99-
def self.filter_sensitive_post_data_parameters(controller, parameters)
100-
exclude_raw_post_parameters?(controller) ? controller.__send__(:filter_parameters, parameters) : parameters
101-
end
102-
103-
def self.filter_sensitive_post_data_from_env(env_key, env_value, controller)
104-
return env_value unless exclude_raw_post_parameters?
105-
return PARAM_FILTER_REPLACEMENT if (env_key =~ /RAW_POST_DATA/i)
106-
return controller.__send__(:filter_parameters, {env_key => env_value}).values[0]
107-
end
10895
end

0 commit comments

Comments
 (0)