airbrake-ruby 3.0.0.rc.4 → 3.0.0.rc.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a2c26bbe2effb0671dd84b762fa9787e8f5bc4cb
-  data.tar.gz: 964f77831dc4162e3268f5714c11ef648c2609f5
+  metadata.gz: 84d9bd8df1233ea2c15f4f299e88724431188ad4
+  data.tar.gz: 30e834bbd33cb36abcd8703b2eaa8b9c3035b6b9
 SHA512:
-  metadata.gz: 4979b96aa750761944b0d778a625801f808dc08cf92b81c722cdf309d0b02986c83b02786e93922ebe1dcf280acc73a500010fb21ca9192c2c709b7bfda286e0
-  data.tar.gz: e0f39ae0d44ab1d13dfc3066caecc1eeedbb21260c3f34a923054b8bcd6240ea6f438e677be59a1dbb12d0fc7ec54e64a1a83b1861a1659bb4f3e2eacc2407e7
+  metadata.gz: 6a6c6bedcb66906b4b8cbe2d3f52b1d2f610e4ef061c5533be301ad01cc9ef12e2f39a58b9758a34b081d74941ae898053cd4cbd44fd164730525f34d7d7957f
+  data.tar.gz: 78b0135645bf7c97265014e86878c0fdc9a67cc2aae2283d07a7f23c349d65996dbd4fd1b081804c0413c9bc53897c19e145e85d97c76176c7e7b610e8333f66

data/lib/airbrake-ruby.rb

@@ -115,7 +115,7 @@ module Airbrake
     def merge_context(_context); end
 
     # @macro see_public_api_method
-    def inc_request(method, route, status_code, dur, time); end
+    def notify_request(request_info); end
   end
 
   # A Hash that holds all notifiers. The keys of the Hash are notifier
@@ -345,26 +345,32 @@ module Airbrake
       @notifiers[:default].merge_context(context)
     end
 
-    # Increments request count of a certain +route+ that was invoked with
-    # +method+, and returned +status_code+ at +time+ and took +dur+
-    # milliseconds.
+    # Increments request statistics of a certain +route+ that was invoked on
+    # +start_time+ and ended on +end_time+ with +method+, and returned
+    # +status_code+.
     #
     # After a certain amount of time (n seconds) the aggregated route
     # information will be sent to Airbrake.
     #
     # @example
-    #   Airbrake.inc_request('POST', '/thing/:id/create', 200, 123, Time.now)
+    #   Airbrake.notify_request(
+    #     method: 'POST',
+    #     route: '/thing/:id/create',
+    #     status_code: 200,
+    #     start_time: timestamp,
+    #     end_time: Time.now
+    #   )
     #
-    # @param [String] method The HTTP method that was invoked
-    # @param [String] route The route that was invoked
-    # @param [Integer] status_code The respose code that the route returned
-    # @param [Float] dur How much time the processing of the request took in
-    #   milliseconds
-    # @param [Time] time When the request happened
+    # @param [Hash{Symbol=>Object}] request_info
+    # @option request_info [String] :method The HTTP method that was invoked
+    # @option request_info [String] :route The route that was invoked
+    # @option request_info [Integer] :status_code The respose code that the route returned
+    # @option request_info [Date] :start_time When the request started
+    # @option request_info [Time] :end_time When the request ended (optional)
     # @return [void]
     # @since v3.0.0
-    def inc_request(method, route, status_code, dur, time)
-      @notifiers[:default].inc_request(method, route, status_code, dur, time)
+    def notify_request(request_info)
+      @notifiers[:default].notify_request(request_info)
     end
   end
 end

data/lib/airbrake-ruby/notifier.rb

@@ -100,8 +100,8 @@ module Airbrake
     end
 
     # @macro see_public_api_method
-    def inc_request(*args)
-      @route_sender.inc_request(*args)
+    def notify_request(request_info)
+      @route_sender.notify_request(request_info)
     end
 
     # @return [String] customized inspect to lessen the amount of clutter

data/lib/airbrake-ruby/route_sender.rb

@@ -87,14 +87,18 @@ module Airbrake
     end
 
     # @macro see_public_api_method
-    def inc_request(method, route, status_code, dur, tm)
-      route = create_route_key(method, route, status_code, tm)
-
+    def notify_request(request_info)
+      route = create_route_key(
+        request_info[:method],
+        request_info[:route],
+        request_info[:status_code],
+        utc_truncate_minutes(request_info[:start_time])
+      )
       promise = Airbrake::Promise.new
 
       @mutex.synchronize do
         @routes[route] ||= RouteStat.new
-        increment_stats(@routes[route], dur)
+        increment_stats(request_info, @routes[route])
 
         if @flush_period > 0
           schedule_flush(promise)
@@ -117,10 +121,10 @@ module Airbrake
       RouteKey.new(method, route, status_code, time.rfc3339)
     end
 
-    def increment_stats(stat, dur)
+    def increment_stats(request_info, stat)
       stat.count += 1
 
-      ms = dur.to_f
+      ms = (request_info[:end_time] || Time.now) - request_info[:start_time]
       stat.sum += ms
       stat.sumsq += ms * ms
 
@@ -159,5 +163,11 @@ module Airbrake
         URI.join(@config.host, "api/v5/projects/#{@config.project_id}/routes-stats")
       )
     end
+
+    def utc_truncate_minutes(time)
+      time_array = time.to_a
+      time_array[0] = 0
+      Time.utc(*time_array)
+    end
   end
 end

data/lib/airbrake-ruby/version.rb

@@ -2,5 +2,5 @@
 # More information: http://semver.org/
 module Airbrake
   # @return [String] the library version
-  AIRBRAKE_RUBY_VERSION = '3.0.0.rc.4'.freeze
+  AIRBRAKE_RUBY_VERSION = '3.0.0.rc.5'.freeze
 end

data/spec/airbrake_spec.rb

@@ -111,13 +111,17 @@ RSpec.describe Airbrake do
     end
   end
 
-  describe ".inc_request" do
-    it "forwards 'inc_request' to the notifier" do
-      t = Time.now
-      expect(default_notifier).to receive(:inc_request).with(
-        'GET', '/foo', 200, 1000, t
-      )
-      described_class.inc_request('GET', '/foo', 200, 1000, t)
+  describe ".notify_request" do
+    it "forwards 'notify_request' to the notifier" do
+      params = {
+        method: 'GET',
+        route: '/foo',
+        status_code: 200,
+        start_time: Time.new(2018, 1, 1, 0, 20, 0, 0),
+        end_time: Time.new(2018, 1, 1, 0, 19, 0, 0)
+      }
+      expect(default_notifier).to receive(:notify_request).with(params)
+      described_class.notify_request(params)
     end
   end
 end

data/spec/notifier_spec.rb

@@ -450,13 +450,18 @@ RSpec.describe Airbrake::Notifier do
     end
   end
 
-  describe "#inc_request" do
-    it "forwards 'inc_request' to RouteSender" do
-      t = Time.now
-      expect_any_instance_of(Airbrake::RouteSender).to receive(:inc_request).with(
-        'GET', '/foo', 200, 1000, t
-      )
-      subject.inc_request('GET', '/foo', 200, 1000, t)
+  describe "#notify_request" do
+    it "forwards 'notify_request' to RouteSender" do
+      params = {
+        method: 'GET',
+        route: '/foo',
+        status_code: 200,
+        start_time: Time.new(2018, 1, 1, 0, 20, 0, 0),
+        end_time: Time.new(2018, 1, 1, 0, 19, 0, 0)
+      }
+      expect_any_instance_of(Airbrake::RouteSender)
+        .to receive(:notify_request).with(params)
+      subject.notify_request(params)
     end
   end
 

data/spec/route_sender_spec.rb

@@ -13,7 +13,7 @@ RSpec.describe Airbrake::RouteSender do
 
   subject { described_class.new(config) }
 
-  describe "#inc_request" do
+  describe "#notify_request" do
     before do
       stub_request(:put, endpoint).to_return(status: 200, body: '')
     end
@@ -22,7 +22,12 @@ RSpec.describe Airbrake::RouteSender do
     after { sleep 0.2 }
 
     it "rounds time to the floor minute" do
-      subject.inc_request('GET', '/foo', 200, 24, Time.new(2018, 1, 1, 0, 0, 20, 0))
+      subject.notify_request(
+        method: 'GET',
+        route: '/foo',
+        status_code: 200,
+        start_time: Time.new(2018, 1, 1, 0, 0, 20, 0)
+      )
       sleep 0.2
       expect(
         a_request(:put, endpoint).with(body: /"time":"2018-01-01T00:00:00\+00:00"/)
@@ -30,8 +35,18 @@ RSpec.describe Airbrake::RouteSender do
     end
 
     it "increments routes with the same key" do
-      subject.inc_request('GET', '/foo', 200, 24, Time.new(2018, 1, 1, 0, 0, 20, 0))
-      subject.inc_request('GET', '/foo', 200, 24, Time.new(2018, 1, 1, 0, 0, 50, 0))
+      subject.notify_request(
+        method: 'GET',
+        route: '/foo',
+        status_code: 200,
+        start_time: Time.new(2018, 1, 1, 0, 0, 20, 0)
+      )
+      subject.notify_request(
+        method: 'GET',
+        route: '/foo',
+        status_code: 200,
+        start_time: Time.new(2018, 1, 1, 0, 0, 50, 0)
+      )
       sleep 0.2
       expect(
         a_request(:put, endpoint).with(body: /"count":2/)
@@ -39,45 +54,74 @@ RSpec.describe Airbrake::RouteSender do
     end
 
     it "groups routes by time" do
-      subject.inc_request('GET', '/foo', 200, 24, Time.new(2018, 1, 1, 0, 0, 20, 0))
-      subject.inc_request('GET', '/foo', 200, 10, Time.new(2018, 1, 1, 0, 1, 20, 0))
+      subject.notify_request(
+        method: 'GET',
+        route: '/foo',
+        status_code: 200,
+        start_time: Time.new(2018, 1, 1, 0, 0, 49, 0),
+        end_time: Time.new(2018, 1, 1, 0, 0, 50, 0)
+      )
+      subject.notify_request(
+        method: 'GET',
+        route: '/foo',
+        status_code: 200,
+        start_time: Time.new(2018, 1, 1, 0, 1, 49, 0),
+        end_time: Time.new(2018, 1, 1, 0, 1, 55, 0)
+      )
       sleep 0.2
       expect(
         a_request(:put, endpoint).with(
           body: %r|\A
             {"routes":\[
               {"method":"GET","route":"/foo","status_code":200,
-               "time":"2018-01-01T00:00:00\+00:00","count":1,"sum":24.0,
-               "sumsq":576.0,"tdigest":"AAAAAkA0AAAAAAAAAAAAAUHAAAAB"},
+               "time":"2018-01-01T00:00:00\+00:00","count":1,"sum":1.0,
+               "sumsq":1.0,"tdigest":"AAAAAkA0AAAAAAAAAAAAAT\+AAAAB"},
               {"method":"GET","route":"/foo","status_code":200,
-               "time":"2018-01-01T00:01:00\+00:00","count":1,"sum":10.0,
-               "sumsq":100.0,"tdigest":"AAAAAkA0AAAAAAAAAAAAAUEgAAAB"}\]}
+               "time":"2018-01-01T00:01:00\+00:00","count":1,"sum":6.0,
+               "sumsq":36.0,"tdigest":"AAAAAkA0AAAAAAAAAAAAAUDAAAAB"}\]}
           \z|x
         )
       ).to have_been_made
     end
 
     it "groups routes by route key" do
-      subject.inc_request('GET', '/foo', 200, 24, Time.new(2018, 1, 1, 0, 0, 20, 0))
-      subject.inc_request('POST', '/foo', 200, 10, Time.new(2018, 1, 1, 0, 0, 20, 0))
+      subject.notify_request(
+        method: 'GET',
+        route: '/foo',
+        status_code: 200,
+        start_time: Time.new(2018, 1, 1, 0, 49, 0, 0),
+        end_time: Time.new(2018, 1, 1, 0, 50, 0, 0)
+      )
+      subject.notify_request(
+        method: 'POST',
+        route: '/foo',
+        status_code: 200,
+        start_time: Time.new(2018, 1, 1, 0, 49, 0, 0),
+        end_time: Time.new(2018, 1, 1, 0, 50, 0, 0)
+      )
       sleep 0.2
       expect(
         a_request(:put, endpoint).with(
           body: %r|\A
             {"routes":\[
               {"method":"GET","route":"/foo","status_code":200,
-               "time":"2018-01-01T00:00:00\+00:00","count":1,"sum":24.0,
-               "sumsq":576.0,"tdigest":"AAAAAkA0AAAAAAAAAAAAAUHAAAAB"},
+               "time":"2018-01-01T00:49:00\+00:00","count":1,"sum":60.0,
+               "sumsq":3600.0,"tdigest":"AAAAAkA0AAAAAAAAAAAAAUJwAAAB"},
               {"method":"POST","route":"/foo","status_code":200,
-               "time":"2018-01-01T00:00:00\+00:00","count":1,"sum":10.0,
-               "sumsq":100.0,"tdigest":"AAAAAkA0AAAAAAAAAAAAAUEgAAAB"}\]}
+               "time":"2018-01-01T00:49:00\+00:00","count":1,"sum":60.0,
+               "sumsq":3600.0,"tdigest":"AAAAAkA0AAAAAAAAAAAAAUJwAAAB"}\]}
           \z|x
         )
       ).to have_been_made
     end
 
     it "returns a promise" do
-      promise = subject.inc_request('GET', '/foo', 200, 24, Time.new)
+      promise = subject.notify_request(
+        method: 'GET',
+        route: '/foo',
+        status_code: 200,
+        start_time: Time.new(2018, 1, 1, 0, 49, 0, 0)
+      )
       sleep 0.2
       expect(promise).to be_an(Airbrake::Promise)
       expect(promise.value).to eq('' => nil)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: airbrake-ruby
 version: !ruby/object:Gem::Version
-  version: 3.0.0.rc.4
+  version: 3.0.0.rc.5
 platform: ruby
 authors:
 - Airbrake Technologies, Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-11-06 00:00:00.000000000 Z
+date: 2018-11-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: tdigest