airbrake-ruby 2.1.0 → 2.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d83d667d85784f12930d891e4a40f33ff5021d32
-  data.tar.gz: 27abbb08a24e1f5037134d779e1cd00bb6b2c6b9
+  metadata.gz: 85e069b4c88a953affccceafa0e250aee3c34367
+  data.tar.gz: 349b91844712169716c97ba606cdebf6e200e908
 SHA512:
-  metadata.gz: 97bde22452bbf79d34d127de8972799f482ee9476a02760de6f9089d02e548a75fa73657f02dd1a93e99ee3ae27b85f176b00d9c362b3d78c837dc49ae2ab6d6
-  data.tar.gz: 9aa4f9c47307cf45339ebabfad298e1217498318029a5423e47a2f4f91d63b14022a41f39940bccd1b67de4f5f54fae8a2dc4dd00b97912ad2c454bfcc60d63e
+  metadata.gz: d4fdabb0447a4c8b0ca06a45cb8ce371d35805c191253edec1bda3fd0581f60ae5382efbf7d64b56f5845d9444fcd145c73844c6dba94abbcff5b94aad00de8f
+  data.tar.gz: 4662128a6ad57a91659539296229fe7c0b824a6049abb32f64f9ab883be919a4ced5af56f38a3c768016b4477844eef5db30a600dceca0daa9c9e9822af973ef

data/lib/airbrake-ruby.rb

@@ -153,10 +153,13 @@ module Airbrake
     # @param [Hash] params The additional payload to be sent to Airbrake. Can
     #   contain any values. The provided values will be displayed in the Params
     #   tab in your project's dashboard
+    # @yield [notice] The notice to filter
+    # @yieldparam [Airbrake::Notice]
+    # @yieldreturn [void]
     # @return [Airbrake::Promise]
     # @see .notify_sync
-    def notify(exception, params = {})
-      @notifiers[:default].notify(exception, params)
+    def notify(exception, params = {}, &block)
+      @notifiers[:default].notify(exception, params, &block)
     end
 
     ##
@@ -166,10 +169,18 @@ module Airbrake
     #   Airbrake.notify_sync('App crashed!')
     #   #=> {"id"=>"123", "url"=>"https://airbrake.io/locate/321"}
     #
+    # @param [Exception, String, Airbrake::Notice] exception The exception to be
+    #   sent to Airbrake
+    # @param [Hash] params The additional payload to be sent to Airbrake. Can
+    #   contain any values. The provided values will be displayed in the Params
+    #   tab in your project's dashboard
+    # @yield [notice] The notice to filter
+    # @yieldparam [Airbrake::Notice]
+    # @yieldreturn [void]
     # @return [Hash{String=>String}] the reponse from the server
     # @see .notify
-    def notify_sync(exception, params = {})
-      @notifiers[:default].notify_sync(exception, params)
+    def notify_sync(exception, params = {}, &block)
+      @notifiers[:default].notify_sync(exception, params, &block)
     end
 
     ##

data/lib/airbrake-ruby/notice.rb

@@ -56,6 +56,10 @@ module Airbrake
     # @return [String] the name of the host machine
     HOSTNAME = Socket.gethostname.freeze
 
+    ##
+    # @return [String]
+    DEFAULT_SEVERITY = 'error'.freeze
+
     ##
     # @since v1.7.0
     # @return [Hash{Symbol=>Object}] the hash with arbitrary objects to be used
@@ -158,7 +162,9 @@ module Airbrake
         environment: @config.environment,
 
         # Make sure we always send hostname.
-        hostname: HOSTNAME
+        hostname: HOSTNAME,
+
+        severity: DEFAULT_SEVERITY
       }.merge(CONTEXT).delete_if { |_key, val| val.nil? || val.empty? }
     end
 

data/lib/airbrake-ruby/notifier.rb

@@ -47,14 +47,14 @@ module Airbrake
 
     ##
     # @macro see_public_api_method
-    def notify(exception, params = {})
-      send_notice(exception, params, default_sender)
+    def notify(exception, params = {}, &block)
+      send_notice(exception, params, default_sender, &block)
     end
 
     ##
     # @macro see_public_api_method
-    def notify_sync(exception, params = {})
-      send_notice(exception, params, @sync_sender).value
+    def notify_sync(exception, params = {}, &block)
+      send_notice(exception, params, @sync_sender, &block).value
     end
 
     ##
@@ -119,6 +119,8 @@ module Airbrake
 
       notice = build_notice(exception, params)
       @filter_chain.refine(notice)
+      yield notice if block_given?
+
       if notice.ignored?
         return promise.reject("#{notice} was marked as ignored")
       end

data/lib/airbrake-ruby/version.rb

@@ -4,5 +4,5 @@
 module Airbrake
   ##
   # @return [String] the library version
-  AIRBRAKE_RUBY_VERSION = '2.1.0'.freeze
+  AIRBRAKE_RUBY_VERSION = '2.2.0'.freeze
 end

data/spec/airbrake_spec.rb

@@ -41,6 +41,19 @@ RSpec.describe Airbrake do
       sleep 2
       expect(a_request(:post, endpoint)).to have_been_made.once
     end
+
+    it "yields a notice" do
+      described_class.notify('bongo') do |notice|
+        notice[:params][:bingo] = :bango
+      end
+
+      sleep 1
+
+      expect(
+        a_request(:post, endpoint).
+        with(body: /params":{.*"bingo":"bango".*}/)
+      ).to have_been_made.once
+    end
   end
 
   describe ".notify_sync" do
@@ -51,6 +64,17 @@ RSpec.describe Airbrake do
       expect(a_request(:post, endpoint)).to have_been_made.once
     end
 
+    it "yields a notice" do
+      described_class.notify_sync('bongo') do |notice|
+        notice[:params][:bingo] = :bango
+      end
+
+      expect(
+        a_request(:post, endpoint).
+        with(body: /params":{.*"bingo":"bango".*}/)
+      ).to have_been_made.once
+    end
+
     describe "clean backtrace" do
       shared_examples 'backtrace building' do |msg, argument|
         it(msg) do

data/spec/notice_spec.rb

@@ -200,6 +200,10 @@ RSpec.describe Airbrake::Notice do
       expect(notice.to_json).
         to match(/"context":{.*"hostname":".+".*}/)
     end
+
+    it "defaults to the error severity" do
+      expect(notice.to_json).to match(/"context":{.*"severity":"error".*}/)
+    end
   end
 
   describe "#[]" do

data/spec/notifier_spec.rb

@@ -378,6 +378,13 @@ RSpec.describe Airbrake::Notifier do
         end
       end
     end
+
+    describe "block argument" do
+      it "yields a notice" do
+        @airbrake.notify_sync(ex) { |notice| notice[:params][:bingo] = :bango }
+        expect_a_request_with_body(/params":{.*"bingo":"bango".*}/)
+      end
+    end
   end
 
   describe "#notify" do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: airbrake-ruby
 version: !ruby/object:Gem::Version
-  version: 2.1.0
+  version: 2.2.0
 platform: ruby
 authors:
 - Airbrake Technologies, Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-04-27 00:00:00.000000000 Z
+date: 2017-05-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec