sjs/grape_logging
1
0
Fork 0
mirror of https://github.com/samsonjs/grape_logging.git synced 2026-03-25 08:55:47 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge pull request #22 from guizmaii/master

Browse source 
Small code simplications
This commit is contained in:
Adrian Serafin 2016-03-08 13:15:10 +01:00
commit 876ddf37ff
2 changed files with 5 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
lib/grape_logging/loggers/filter_parameters.rb
View file

@ -12,13 +12,12 @@ module GrapeLogging
private
def replace_parameters(parameters)
parameters.tap do |parameters|
@filter_parameters.each do |parameter_name|
if parameters.key?(parameter_name.to_s)
parameters[parameter_name.to_s] = @replacement
end
@filter_parameters.each do |parameter_name|
if parameters.key?(parameter_name.to_s)
parameters[parameter_name.to_s] = @replacement
end
end
parameters
end
end
end

6
lib/grape_logging/middleware/request_logger.rb
View file

@ -29,12 +29,8 @@ module GrapeLogging
def after
stop_time
params = collect_parameters
@reporter.perform(params)
@reporter.perform(collect_parameters)
invoke_included_loggers(:after)
nil
end