datadog-notifications 0.4.4 → 0.4.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 272c499c77016ad164fc1ab11d9daf1e4fd2030e
-  data.tar.gz: 453640237e120a83ff833d9b9dc311349e0fa175
+  metadata.gz: 48de82448ce774595bad633515891226c23c33d6
+  data.tar.gz: 717fdf4a07d04db7d39043d814bf508bcabd44fa
 SHA512:
-  metadata.gz: 5d3630229b5be58bc741717488414e57dbb5cbc270b275be3f5717d7a725489d5f087e2391182b798d9c08a4ca116f57691a92d61c8c501c9d04e93424dc5d90
-  data.tar.gz: c1520297c7612fb23e4e59db32979302e069a91c49f6c9f1a1a15256f5ab91167c8f17f586ce6234ea9030232132253b97d135bd6fee194f1cd3a3233c8116d4
+  metadata.gz: d2d93bacaa972855f6776ea639a70294ee3e8258cd55fb8584e1495b13ff8722fc64d9c1168acdbb0897b63f8cf5e2ce4ed29400e5839bca6a809a04e4fb3f48
+  data.tar.gz: 7c5ade1127be0fca2f56f622806489bd267e9e4aee2b368f24a7f952d45605def26c0877f2384bd639032ae770636d7c4582980934ce1e9bc56d9682c02b8a6f

data/Gemfile.lock

@@ -1,20 +1,20 @@
 PATH
   remote: .
   specs:
-    datadog-notifications (0.4.4)
+    datadog-notifications (0.4.5)
       activesupport
       dogstatsd-ruby (~> 2.0.0)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    activemodel (5.0.1)
-      activesupport (= 5.0.1)
-    activerecord (5.0.1)
-      activemodel (= 5.0.1)
-      activesupport (= 5.0.1)
+    activemodel (5.0.2)
+      activesupport (= 5.0.2)
+    activerecord (5.0.2)
+      activemodel (= 5.0.2)
+      activesupport (= 5.0.2)
       arel (~> 7.0)
-    activesupport (5.0.1)
+    activesupport (5.0.2)
       concurrent-ruby (~> 1.0, >= 1.0.2)
       i18n (~> 0.7)
       minitest (~> 5.1)
@@ -24,16 +24,16 @@ GEM
       descendants_tracker (~> 0.0.4)
       ice_nine (~> 0.11.0)
       thread_safe (~> 0.3, >= 0.3.1)
-    builder (3.2.2)
+    builder (3.2.3)
     coercible (1.0.0)
       descendants_tracker (~> 0.0.1)
-    concurrent-ruby (1.0.4)
+    concurrent-ruby (1.0.5)
     descendants_tracker (0.0.4)
       thread_safe (~> 0.3, >= 0.3.1)
-    diff-lcs (1.2.5)
+    diff-lcs (1.3)
     dogstatsd-ruby (2.0.0)
     equalizer (0.0.11)
-    grape (0.19.0)
+    grape (0.19.1)
       activesupport
       builder
       hashie (>= 2.1.0)
@@ -43,8 +43,8 @@ GEM
       rack (>= 1.3.0)
       rack-accept
       virtus (>= 1.0.0)
-    hashie (3.4.6)
-    i18n (0.7.0)
+    hashie (3.5.5)
+    i18n (0.8.1)
     ice_nine (0.11.2)
     minitest (5.10.1)
     multi_json (1.12.1)
@@ -72,8 +72,8 @@ GEM
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.5.0)
     rspec-support (3.5.0)
-    sqlite3 (1.3.12)
-    thread_safe (0.3.5)
+    sqlite3 (1.3.13)
+    thread_safe (0.3.6)
     tool (0.2.3)
     tzinfo (1.2.2)
       thread_safe (~> 0.1)
@@ -97,4 +97,4 @@ DEPENDENCIES
   sqlite3
 
 BUNDLED WITH
-   1.11.2
+   1.14.5

data/lib/datadog/notifications/plugins/grape.rb

@@ -1,15 +1,17 @@
 module Datadog::Notifications::Plugins
   class Grape < Base
 
-    attr_reader :metric_name
+    attr_reader :metric_name, :exception_handler
 
     # Options:
     #
     # *<tt>:metric_name</tt> - the metric name, defaults to "grape.request"
+    # *<tt>:exception_handler</tt> - a custom exception handler proc which accepts an exception object and returns a status
     # *<tt>:tags</tt> - additional tags
     def initialize(opts = {})
       super
       @metric_name = opts[:metric_name] || "grape.request"
+      @exception_handler = opts[:exception_handler] || ->_ { 500 }
 
       Datadog::Notifications.subscribe 'endpoint_run.grape' do |reporter, event|
         record reporter, event
@@ -24,6 +26,11 @@ module Datadog::Notifications::Plugins
       route    = endpoint.route
       version  = route.version
       method   = route.request_method
+      status   = endpoint.status
+
+      if payload[:exception_object]
+        status = exception_handler.call(payload[:exception_object])
+      end
 
       path = route.pattern.path.dup
       path.sub!(/\(\.\:format\)$/, '')
@@ -31,7 +38,7 @@ module Datadog::Notifications::Plugins
       path.gsub!(/:(\w+)/) {|m| m[1..-1].upcase }
       path.gsub!(/[^\w\/\-]+/, '_')
 
-      tags = self.tags + %W|method:#{method} path:#{path} status:#{endpoint.status}|
+      tags = self.tags + %W|method:#{method} path:#{path} status:#{status}|
       tags.push "version:#{version}" if version
 
       reporter.batch do

data/lib/datadog/notifications/version.rb

@@ -1,5 +1,5 @@
 module Datadog
   class Notifications
-    VERSION = "0.4.4"
+    VERSION = "0.4.5"
   end
 end

data/spec/datadog/notifications/plugins/grape_spec.rb

@@ -4,6 +4,8 @@ describe Datadog::Notifications::Plugins::Grape do
   include Rack::Test::Methods
 
   let(:app) do
+    unauthorized = Class.new(RuntimeError)
+
     sub_api = Class.new(Grape::API) do
       version 'v1'
       prefix  'api'
@@ -12,10 +14,19 @@ describe Datadog::Notifications::Plugins::Grape do
     end
 
     Class.new(Grape::API) do
+
+      rescue_from unauthorized do |e|
+        error!({ message: "unauthorized", error: '401 Unauthorized' }, 401)
+      end
+
       get 'echo/:key1/:key2' do
         "#{params['key1']} #{params['key2']}"
       end
 
+      get '/rescued' do
+        raise unauthorized.new("unauthorized")
+      end
+
       namespace :sub do
         mount sub_api
 
@@ -59,4 +70,21 @@ describe Datadog::Notifications::Plugins::Grape do
     ])
   end
 
+  it 'should handle rescued errors' do
+    get '/rescued'
+    expect(last_response.status).to eq(401)
+
+    expect(buffered).to eq([
+      "api.request:1|c|#custom:tag,env:test,host:test.host,more:tags,method:GET,path:/rescued,status:401",
+      "api.request.time:333|ms|#custom:tag,env:test,host:test.host,more:tags,method:GET,path:/rescued,status:401",
+    ])
+  end
+
+  it 'should not report paths on 404s' do
+    get '/sub/missing'
+    expect(last_response.status).to eq(404)
+
+    expect(buffered).to eq([])
+  end
+
 end

data/spec/spec_helper.rb

@@ -62,5 +62,8 @@ Datadog::Notifications.configure do |c|
   c.tags     = ["custom:tag"]
 
   c.use Datadog::Notifications::Plugins::ActiveRecord
-  c.use Datadog::Notifications::Plugins::Grape, tags: ["more:tags"], metric_name: "api.request"
+  c.use Datadog::Notifications::Plugins::Grape,
+    tags: ["more:tags"],
+    metric_name: "api.request",
+    exception_handler: ->e { e.message.include?("unauthorized") ? 401 : 500 }
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: datadog-notifications
 version: !ruby/object:Gem::Version
-  version: 0.4.4
+  version: 0.4.5
 platform: ruby
 authors:
 - Dimitrij Denissenko
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-01-03 00:00:00.000000000 Z
+date: 2017-03-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -187,7 +187,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.1
+rubygems_version: 2.6.8
 signing_key: 
 specification_version: 4
 summary: Generic ActiveSupport::Notifications Datadog handler