salestation 3.2.1 → 3.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a8e5a8bb08a8757fd7a3cd570d3f168907fba0dfd98ba86a13b192c401e901a2
-  data.tar.gz: 0a76d752b8b69f67ead6e758f8084faf7645ccb5a88a994dcbd693f0fd7da6f3
+  metadata.gz: f2ce01f52dfc5a837a48d3ce6923fb435edc830262506666a368c93f96e6ad83
+  data.tar.gz: 899d05be06cafb0164abafcce4958f1fea6abe48b0ea118c6b44206be1dbd865
 SHA512:
-  metadata.gz: c14ccd4396d3aee053517a703b759928e511e8b11ba33ed6eb0e1cf72244b2eaf905fa8adcc8bed6f58b68f82e896e38e3646ef3803c92302112750232b319d0
-  data.tar.gz: 2a3ebef7fc8abe6f2a29178382a872297348d10afef9d9c3edc3aeecb658fe9ad8dd09d1cb042d8fcad4451bac417fb58a174e457f8863f4be05adf4ef68c540
+  metadata.gz: '0109c1a56a529c85a8f252425524d6d4012c18e5da7fb729c070c2de8c3a716a09f5059bdd7201f47d977c0fa3ee8e060617a8ebafefd00c1d698d90afdcee72'
+  data.tar.gz: c83118eac0a175287c7ce9af30c225aa02bff963b8ec7360a108297a9c076e06916454cb8949ea2ff8cb1da4ee03ff979831f51a5fcc0246376dea3e23a91e3f

data/lib/salestation/web/request_logger.rb

@@ -1,11 +1,9 @@
 # frozen_string_literal: true
 
-require 'json'
-
 module Salestation
   class Web < Module
     class RequestLogger
-
+      DURATION_PRECISION = 6
       REMOTE_ADDR = 'REMOTE_ADDR'
       REQUEST_URI = 'REQUEST_URI'
       REQUEST_METHOD = 'REQUEST_METHOD'
@@ -14,7 +12,6 @@ module Salestation
       HTTP_USER_AGENT = 'HTTP_USER_AGENT'
       HTTP_ACCEPT = 'HTTP_ACCEPT'
       SERVER_NAME = 'SERVER_NAME'
-      JSON_CONTENT_TYPE = 'application/json'
 
       def initialize(app, logger, log_response_body: false)
         @app = app
@@ -23,7 +20,7 @@ module Salestation
       end
 
       def call(env)
-        began_at = Time.now
+        began_at = system_monotonic_time
 
         @app.call(env).tap do |status, headers, body|
           type = status >= 500 ? :error : :info
@@ -44,30 +41,25 @@ module Salestation
           http_accept:  env[HTTP_ACCEPT],
           server_name:  env[SERVER_NAME],
           status:       status,
-          duration:     Time.now - began_at,
+          duration:     duration(from: began_at),
           headers:      headers
         }
 
         if status >= 400
-          log[:error] = parse_body(body)
+          log[:error] = body.join
         elsif @log_response_body
-          log[:body] = parse_body(body)
+          log[:body] = body.join
         end
 
         log
       end
 
-      def parse_body(body)
-        # Rack body is an array
-        return {} if body.empty?
+      def duration(from:)
+        (system_monotonic_time - from).round(DURATION_PRECISION)
+      end
 
-        if defined?(Oj)
-          Oj.load(body.join)
-        else
-          JSON.parse(body.join)
-        end
-      rescue Exception
-        {error: 'Failed to parse response body'}
+      def system_monotonic_time
+        Process.clock_gettime(Process::CLOCK_MONOTONIC)
       end
     end
   end

data/salestation.gemspec

@@ -4,7 +4,7 @@ $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 
 Gem::Specification.new do |spec|
   spec.name          = "salestation"
-  spec.version       = "3.2.1"
+  spec.version       = "3.2.2"
   spec.authors       = ["SaleMove TechMovers"]
   spec.email         = ["techmovers@salemove.com"]
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: salestation
 version: !ruby/object:Gem::Version
-  version: 3.2.1
+  version: 3.2.2
 platform: ruby
 authors:
 - SaleMove TechMovers
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-10-09 00:00:00.000000000 Z
+date: 2019-11-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler