sjs/grape_logging
1
0
Fork 0
mirror of https://github.com/samsonjs/grape_logging.git synced 2026-04-07 11:05:47 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

clean up middleware code & unsubscribe in the end

Browse source
This commit is contained in:
Vladimir Kochnev 2015-06-26 00:04:53 +03:00
parent d130386947
commit 296705e02e
1 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -7,7 +7,7 @@ module GrapeLogging
start_time
@db_duration = 0
ActiveSupport::Notifications.subscribe('sql.active_record') do |*args|
@subscription = ActiveSupport::Notifications.subscribe('sql.active_record') do |*args|
event = ActiveSupport::Notifications::Event.new(*args)
@db_duration += event.duration
end if defined?(ActiveRecord)
@ -15,12 +15,13 @@ module GrapeLogging
def after
stop_time
logger.info parameters(request, response)
logger.info parameters
ActiveSupport::Notifications.unsubscribe(@subscription) if @subscription
nil
end
protected
def parameters(request, response)
def parameters
{
path: request.path,
params: request.params.to_hash,