logjam_agent 0.33.0 → 0.33.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: dbd400c4a9aedda3d5659a550bd2da21e9a29696cd1e5c77c14cb1f15d8e7e50
-  data.tar.gz: d71e179fd331b16c0b2117a31a15e45278477b8f4709ffaf93d91c2a859d9a44
+  metadata.gz: 29e2045e24e8a76d6cd8d13c4521562f237849880f39eec65c6be243b275cd06
+  data.tar.gz: 264a22f43eafaca127977d689ede51a657c4aa9403f5339528c56647f0a502c7
 SHA512:
-  metadata.gz: d5fbde19a0225c1859940ab1add7a690ef67109f861154e7a9d80bbc33834ae18fc415d1ab90ca891d7c245c20761c96ab92d07e99ca011ac5866a740ba37376
-  data.tar.gz: ff9c7d40001b7902f5fbccb8db57b32d57f23760e74225412be354d046967c97c7e0eafec4bc31b7cd2c961920d31fbc2645671091081e441100001ce82c1d40
+  metadata.gz: 4fb90fd911820483409d59f8b0d8f272fe06b55460127899edbfc77da57cf064b32c633dee61bd1bf113acbf205487689bfdc43d623c6d8faaea4bbff9523bd5
+  data.tar.gz: 86470089e012dbf3462725708e65e0deb3a5d8d9cb1268452fb051948d80f55ce6c2361ffde3064aff41f8c9049a5d3452cfad4cbb8c7e87e455142289aed8f2

data/README.md

@@ -10,7 +10,8 @@ Has experimental support for Sinatra.
 Currently only one mechanism is available for data transport:
 ZeroMQ. Support for AMQP has been dropped.
 
-[![Travis](https://travis-ci.org/skaes/logjam_agent.svg?branch=master)](https://travis-ci.org/github/skaes/logjam_agent)
+
+![Build](https://github.com/skaes/logjam_agent/actions/workflows/run-tests.yml/badge.svg)
 
 
 ## Usage

data/lib/logjam_agent/rack/logger.rb

@@ -32,23 +32,16 @@ module LogjamAgent
 
       def call_app(request, env)
         start_time = Time.now
-        start_time_header = env['HTTP_X_STARTTIME']
-        if start_time_header
-          if start_time_header =~ /\At=(\d+)\z/
-            # HTTP_X_STARTTIME is microseconds since the epoch (UTC)
-            http_start_time = Time.at($1.to_f / 1_000_000.0)
-          elsif start_time_header =~ /\Ats=(\d+)(?:\.(\d+))?\z/
-            # HTTP_X_STARTTIME is seconds since the epoch (UTC) with a milliseconds resolution
-            http_start_time = Time.at($1.to_f + $2.to_f / 1000)
-          end
-
-          if http_start_time && (wait_time_ms = (start_time - http_start_time) * 1000) > 0
+        wait_time_ms = 0.0
+        if http_start_time = extract_http_start_time(env)
+          wait_time_ms = (start_time - http_start_time) * 1000
+          if wait_time_ms > 0
             start_time = http_start_time
+          else
+            wait_time_ms = 0.0
           end
-        else
-          wait_time_ms = 0.0
         end
-        before_dispatch(request, env, start_time)
+        before_dispatch(request, env, start_time, wait_time_ms)
         result = @app.call(env)
       rescue ActionDispatch::RemoteIp::IpSpoofAttackError
         result = [403, {}, ['Forbidden']]
@@ -57,6 +50,20 @@ module LogjamAgent
         after_dispatch(env, result, run_time_ms, wait_time_ms)
       end
 
+      def extract_http_start_time(env)
+        start_time_header = env['HTTP_X_STARTTIME']
+        if start_time_header
+          if start_time_header =~ /\At=(\d+)\z/
+            # HTTP_X_STARTTIME is microseconds since the epoch (UTC)
+            return Time.at($1.to_f / 1_000_000.0)
+          elsif start_time_header =~ /\Ats=(\d+)(?:\.(\d+))?\z/
+            # HTTP_X_STARTTIME is seconds since the epoch (UTC) with a milliseconds resolution
+            return Time.at($1.to_f + $2.to_f / 1000)
+          end
+        end
+        return nil
+      end
+
       def compute_tags(request)
         @taggers.collect do |tag|
           case tag
@@ -78,7 +85,7 @@ module LogjamAgent
         false
       end
 
-      def before_dispatch(request, env, start_time)
+      def before_dispatch(request, env, start_time, wait_time_ms)
         logger.formatter.reset_attributes if logger.formatter.respond_to?(:reset_attributes)
         TimeBandits.reset
         Thread.current.thread_variable_set(:time_bandits_completed_info, nil)
@@ -90,6 +97,7 @@ module LogjamAgent
 
         logjam_request.start_time = start_time
         logjam_fields = logjam_request.fields
+        logjam_fields[:wait_time] = wait_time_ms if wait_time_ms > 0.0
         spoofed = nil
         ip = nil
         begin
@@ -125,7 +133,6 @@ module LogjamAgent
 
         if wait_time_ms < 0
           warn LogjamAgent::NegativeWaitTime.new("#{wait_time_ms} ms")
-          wait_time_ms = 0.0
         end
 
         message = "Completed #{status} #{::Rack::Utils::HTTP_STATUS_CODES[status]} in %.1fms" % run_time_ms
@@ -135,7 +142,6 @@ module LogjamAgent
         ActiveSupport::LogSubscriber.flush_all!
         request_info = { :total_time => run_time_ms, :code => status }
         request_info[:view_time] = view_time if view_time
-        request_info[:wait_time] = wait_time_ms if wait_time_ms > 0
         logjam_request.fields.merge!(request_info)
 
         env["time_bandits.metrics"] = TimeBandits.metrics

data/lib/logjam_agent/version.rb

@@ -1,3 +1,3 @@
 module LogjamAgent
-  VERSION = "0.33.0"
+  VERSION = "0.33.1"
 end

data/test/sinatra_test.rb

@@ -37,6 +37,7 @@ module LogjamAgent
       assert_kind_of String, payload["started_at"]
       assert_kind_of Integer, payload["started_ms"]
       assert_kind_of String, payload["ip"]
+      assert_nil payload["wait_time"]
       # assert_kind_of Float, payload["view_time"]
       lines = payload["lines"]
       assert_match(/Started GET.*password=\[FILTERED\]/, lines[0][2])

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: logjam_agent
 version: !ruby/object:Gem::Version
-  version: 0.33.0
+  version: 0.33.1
 platform: ruby
 authors:
 - Stefan Kaes
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-08-07 00:00:00.000000000 Z
+date: 2022-03-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -275,7 +275,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.24
+rubygems_version: 3.3.4
 signing_key:
 specification_version: 4
 summary: Logjam client library to be used with logjam