Since the Grape v0.14.1, the after callback is always called, so response could be nil if the called fail with an exception

This commit is contained in:
jules 2016-02-29 16:20:40 +01:00
parent fe72aa1a15
commit ef946bf7fe
2 changed files with 8 additions and 4 deletions

View file

@ -2,9 +2,13 @@ module GrapeLogging
module Loggers module Loggers
class Response < GrapeLogging::Loggers::Base class Response < GrapeLogging::Loggers::Base
def parameters(request, response) def parameters(request, response)
if response
{ {
response: serialized_response_body(response) response: serialized_response_body(response)
} }
else
{}
end
end end
private private

View file

@ -51,7 +51,7 @@ module GrapeLogging
protected protected
def parameters def parameters
{ {
status: response.status, status: response.try(:status),
time: { time: {
total: total_runtime, total: total_runtime,
db: db_runtime, db: db_runtime,