inst_statsd 2.1.6 → 2.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cc4aff5a3dabe6fd65126f1b1f80d61cf0e11067608071d0f2429a013abf556e
-  data.tar.gz: 7fc0ed85b4184df2e116fb86b8554d6a74cc476accd4abf2ae1dd06251daca53
+  metadata.gz: 568ce7b0b1164ceb930ba4c27604d3600f3afc9faf9486fa55ed315c3e1b6e90
+  data.tar.gz: e8970517048f627c0449ba8fa241adaae49b9aa37b3a4faa5047aa64e67f07d6
 SHA512:
-  metadata.gz: 2df91d41c5a50fefcb46243e2de9eb7a8dd8caf568786c8ed721e6f46bde1c420e9b85d6b49118217603cc7a202a15a4c07315a4e1bfd6bae4557ccceb581ce0
-  data.tar.gz: a6c7ab6cae3d3e5b19f7fde3da1d52d3d15104f26fb501262e8731804418aabf72cfa3613a3f6a930a56b4507f433507bb5333d74c7f05dc51cb8ac8872ddad7
+  metadata.gz: 92c65a63134c03d227481e0fca2cf23e438268dd8fa59a124ac95795c7e3ec304ac6d7052d7e9e0270cdfdb89ff7bba24543b4021fdfed3dc098fedb6951364a
+  data.tar.gz: 53087ac5aa2d2efe2af74d037ed18a0723774b38e17e3ddb34a8de2b5f67ec74620ef33c02737f3393a1faaf48c81ff5cd0cbfb217f408eb57b249f97ed312d4

data/lib/inst_statsd/block_stat.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module InstStatsd
   class BlockStat
 

data/lib/inst_statsd/block_tracking.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'benchmark'
 
 module InstStatsd

data/lib/inst_statsd/counter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module InstStatsd
   class Counter
     class << self

data/lib/inst_statsd/default_tracking.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "active_support"
 
 module InstStatsd

data/lib/inst_statsd/null_logger.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'logger'
 
 module InstStatsd

data/lib/inst_statsd/request_logger.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module InstStatsd
   class RequestLogger
 

data/lib/inst_statsd/request_stat.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module InstStatsd
   class RequestStat < BlockStat
     def initialize(name, start, finish, id, payload, statsd=InstStatsd::Statsd)
@@ -18,6 +20,7 @@ module InstStatsd
         self.short_stat = "request"
         self.tags[:controller] = controller if controller
         self.tags[:action] = action if action
+        self.tags[:status] = status if status
       else
         self.common_key = "request.#{controller}.#{action}" if controller && action
       end
@@ -46,6 +49,17 @@ module InstStatsd
       @payload.fetch(:params, {})['action']
     end
 
+    def status
+      status = @payload.fetch(:status, 0)
+      # Only return status group to reduce the number of indexed custom metrics (and cost) of datadog
+      return '1XX' if status >= 100 and status < 200
+      return '2XX' if status >= 200 and status < 300
+      return '3XX' if status >= 300 and status < 400
+      return '4XX' if status >= 400 and status < 500
+      return '5XX' if status >= 500 and status < 600
+      nil
+    end
+
     def total
       if (!@finish || !@start)
         return 0

data/lib/inst_statsd/request_tracking.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module InstStatsd
   class RequestTracking
 

data/lib/inst_statsd/sql_tracker.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module InstStatsd
   class SqlTracker
 

data/lib/inst_statsd/statsd.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Proxy class to communicate messages to statsd
 # Available statsd messages are described in:
 #   https://github.com/etsy/statsd/blob/master/README.md

data/lib/inst_statsd.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'statsd'
 
 module InstStatsd

data/spec/inst_statsd/block_stat_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe InstStatsd::BlockStat do

data/spec/inst_statsd/block_tracking_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe InstStatsd::BlockTracking do

data/spec/inst_statsd/counter_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe InstStatsd::Counter do

data/spec/inst_statsd/inst_statsd_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe InstStatsd do

data/spec/inst_statsd/null_logger_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe InstStatsd::NullLogger do

data/spec/inst_statsd/request_logger_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 require 'logger'
 

data/spec/inst_statsd/request_stat_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 
@@ -68,6 +70,22 @@ describe InstStatsd::RequestStat do
     end
   end
 
+  describe '#status' do
+    it 'should return nil if status is not defined' do
+      rs = create_subject
+      expect(rs.status).to eq nil
+    end
+
+    it 'should return HTTP status group if present' do
+      expect(create_subject({status: 200}).status).to eq '2XX'
+      expect(create_subject({status: 201}).status).to eq '2XX'
+      expect(create_subject({status: 302}).status).to eq '3XX'
+      expect(create_subject({status: 400}).status).to eq '4XX'
+      expect(create_subject({status: 404}).status).to eq '4XX'
+      expect(create_subject({status: 503}).status).to eq '5XX'
+    end
+  end
+
   describe '#total' do
     it 'correctly calculates milliseconds from start, finish' do
       rs = create_subject({params: {}})
@@ -111,10 +129,11 @@ describe InstStatsd::RequestStat do
         params: {
           'controller' => 'foo',
           'action'     => 'index'
-        }
+        },
+        status: 200
       }
       rs = create_subject(payload, statsd)
-      expect(statsd).to receive(:timing).with('request.total', 1000, {short_stat: 'request.total', tags: {action: "index", controller: "foo"}} )
+      expect(statsd).to receive(:timing).with('request.total', 1000, {short_stat: 'request.total', tags: {action: "index", controller: "foo", status: '2XX'}} )
       rs.report
     end
 

data/spec/inst_statsd/request_tracking_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe InstStatsd::RequestTracking do

data/spec/inst_statsd/sql_tracker_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 module InstStatsd

data/spec/inst_statsd/statsd_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe InstStatsd::Statsd do

data/spec/spec_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'inst_statsd'
 
 RSpec.configure do |config|

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: inst_statsd
 version: !ruby/object:Gem::Version
-  version: 2.1.6
+  version: 2.3.0
 platform: ruby
 authors:
 - Nick Cloward
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-05-24 00:00:00.000000000 Z
+date: 2022-07-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: dogstatsd-ruby
@@ -43,14 +43,14 @@ dependencies:
   name: aroi
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: 0.0.7
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: 0.0.7
 - !ruby/object:Gem::Dependency
@@ -174,8 +174,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: Statsd for Instructure