grape_logging 1.0.3 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: fcf09ce48949b6fa9b08da15da8aad3bc97cad72
-  data.tar.gz: 5d4206727f908558817d5a8ea41deb4da85f4f25
+  metadata.gz: d214c8c6a482c7b66608429000a8b160e52d896b
+  data.tar.gz: 5c2fa4b0588255709302386f2e5f6070f6b5ad45
 SHA512:
-  metadata.gz: 93a61ce5749bfb9bbb99e2bd1f15701432426f959da03d2b0203d2a12dd74222538b80794b61b39ffb886c2eb061f64b8817a757793ea9265b75f8b05baf57f3
-  data.tar.gz: ab8caacfa27adaab5c0a635e4533cf1de2f349e1a33a9064ae7bbb5fadfb3c16d4f327a0c500977cf3189ea693c8d0cce487dac15e166ef327d814865e0a1637
+  metadata.gz: d692b98646ecf1ae6a5008625ab711737c8ceaa10b91064ac855df009c724350f757d83672a83d43eb826c8b232c2a64882e124e66071950b9f8f3ed359a8fe9
+  data.tar.gz: 0ac0c4faf27ca3318494f97a8535e7d8a0d174fa053bb6003542ccd702108e59553a5a59932858779f4f6fc044ec3a3c47f92c019ed507af9077d2f0aa64dcdc

data/lib/grape_logging/middleware/request_logger.rb

@@ -1,38 +1,35 @@
-require 'benchmark'
 require 'grape/middleware/base'
 
 module GrapeLogging
   module Middleware
     class RequestLogger < Grape::Middleware::Base
       def before
-        env[:db_duration] = 0
+        start_time
 
+        @db_duration = 0
         ActiveSupport::Notifications.subscribe('sql.active_record') do |*args|
           event = ActiveSupport::Notifications::Event.new(*args)
-          env[:db_duration] += event.duration
+          @db_duration += event.duration
         end if defined?(ActiveRecord)
       end
 
-      def call!(env)
-        original_response = nil
-
-        duration = Benchmark.realtime { original_response = super(env) }
-        logger.info parameters(original_response, duration)
-
-        original_response
+      def after
+        stop_time
+        logger.info parameters(request, response)
+        nil
       end
 
       protected
-        def parameters(response, duration)
-          {
-              path: request.path,
-              params: request.params,
-              method: request.request_method,
-              total: (duration * 1000).round(2),
-              db: request.env[:db_duration].round(2),
-              status: response.first
-          }
-        end
+      def parameters(request, response)
+        {
+            path: request.path,
+            params: request.params.to_hash,
+            method: request.request_method,
+            total: total_runtime,
+            db: @db_duration.round(2),
+            status: response.status
+        }
+      end
 
       private
       def logger
@@ -42,6 +39,18 @@ module GrapeLogging
       def request
         @request ||= ::Rack::Request.new(env)
       end
+
+      def total_runtime
+        ((stop_time - start_time) * 1000).round(2)
+      end
+
+      def start_time
+        @start_time ||= Time.now
+      end
+
+      def stop_time
+        @stop_time ||= Time.now
+      end
     end
   end
 end

data/lib/grape_logging/version.rb

@@ -1,3 +1,3 @@
 module GrapeLogging
-  VERSION = '1.0.3'
+  VERSION = '1.1.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: grape_logging
 version: !ruby/object:Gem::Version
-  version: 1.0.3
+  version: 1.1.0
 platform: ruby
 authors:
 - aserafin
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2015-05-04 00:00:00.000000000 Z
+date: 2015-05-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: grape