bugsnag 6.0.0 → 6.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: bdb80b010ab8da05acfddaa542a8b7cbff4be599
-  data.tar.gz: 7fa7188b650b1ebc07e074f4d6f2f1a365105033
+  metadata.gz: 7051f1b045ba90626c2130cda598dcc8db6e93c4
+  data.tar.gz: bf4bd085a0f1cf838d26335f6567128768ef1cde
 SHA512:
-  metadata.gz: 9f19d8a8b92a80747fb160ca3d55c5b19c35aefe8c9293eff1843668a76e90965b37a643e90ebc912b4e1e621411d0968241e024d01b490b5a1d0484621a2b2f
-  data.tar.gz: 2b46521a1dbe916993b13f546992533ef2ab0422cf45a27ea6a30c1c13f8ba01c0a42c2e2486ac655eccff531443452a00fe765e51fea039346d205803da9373
+  metadata.gz: 146a2cdd6ba41c334948ed643263dab8b4480fec92c430bd4f4025704ce3a7089b0d1a9e0778244d0ec79664670090b132aac6e5f4d7e3ab960218dfc7205c53
+  data.tar.gz: 8db4ec587f1c9e0760dc71f569fbb947ee6addf0dc1074b18cc1baddd6d1872fb21a7a314c9194c2a91504a0fcbd5de1ab7da5ca08a9e9c3949aa2de72cafb12

data/CHANGELOG.md

@@ -1,6 +1,11 @@
 Changelog
 =========
 
+## 6.0.1 (09 Nov 2017)
+
+Adds a warning for the change in usage for the `notify()` method from < 6.0 to
+ease upgrading.
+
 ## 6.0.0 (09 Nov 2017)
 
 This notifier has been extensively re-written to make it easier to add new integrations and maintain in the future.  This has led to several changes that are not backwards compatible.  Please refer to the [upgrading guide](https://github.com/bugsnag/bugsnag-ruby/blob/master/UPGRADING.md) for more information.

data/CONTRIBUTING.md

@@ -48,7 +48,8 @@ If you're a member of the core team, follow these instructions for releasing bug
 * Commit/push your changes
 
     ```
-    git commit -am v5.X.X
+    git add VERSION CHANGELOG.md README.md
+    git commit -m "Release v6.x.x"
     git push origin master
     ```
 

data/Gemfile

@@ -3,6 +3,7 @@ source "https://rubygems.org"
 group :test, optional: true do
     gem 'rake', '~> 10.1.1'
     gem 'rspec'
+    gem 'rspec-mocks'
     gem 'rdoc'
     gem 'pry'
     gem 'addressable', '~> 2.3.8'

data/VERSION

@@ -1 +1 @@
-6.0.0
+6.0.1

data/lib/bugsnag.rb

@@ -43,6 +43,10 @@ module Bugsnag
 
     # Explicitly notify of an exception
     def notify(exception, auto_notify=false, &block)
+      unless auto_notify.is_a? TrueClass or auto_notify.is_a? FalseClass
+        configuration.warn("Adding metadata/severity using a hash is no longer supported, please use block syntax instead")
+        auto_notify = false
+      end
 
       if !configuration.auto_notify && auto_notify
         configuration.debug("Not notifying because auto_notify is disabled")
@@ -106,7 +110,6 @@ module Bugsnag
         # Deliver
         configuration.info("Notifying #{configuration.endpoint} of #{report.exceptions.last[:errorClass]}")
         payload_string = ::JSON.dump(Bugsnag::Helpers.trim_if_needed(report.as_json))
-        configuration.debug("Payload: #{payload_string}")
         Bugsnag::Delivery[configuration.delivery_method].deliver(configuration.endpoint, payload_string, configuration)
       end
     end

data/lib/bugsnag/delivery/synchronous.rb

@@ -10,7 +10,7 @@ module Bugsnag
         def deliver(url, body, configuration)
           begin
             response = request(url, body, configuration)
-            configuration.debug("Notification to #{url} finished, response was #{response.code}, payload was #{body}")
+            configuration.debug("Request to #{url} completed, status: #{response.code}")
           rescue StandardError => e
             # KLUDGE: Since we don't re-raise http exceptions, this breaks rspec
             raise if e.class.to_s == "RSpec::Expectations::ExpectationNotMetError"

data/spec/bugsnag_spec.rb

@@ -0,0 +1,19 @@
+# encoding: utf-8
+require 'spec_helper'
+
+describe Bugsnag do
+  describe 'notify' do
+    before do
+      Bugsnag.configuration.logger = spy('logger')
+    end
+
+    it 'does not log an error when sending valid arguments as auto_notify' do
+      notify_test_exception(true)
+      expect(Bugsnag.configuration.logger).to_not have_received(:warn)
+    end
+    it 'logs an error when sending invalid arguments as auto_notify' do
+      notify_test_exception({severity: 'info'})
+      expect(Bugsnag.configuration.logger).to have_received(:warn)
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -12,6 +12,7 @@ require 'bugsnag'
 
 require 'webmock/rspec'
 require 'rspec/expectations'
+require 'rspec/mocks'
 
 class BugsnagTestException < RuntimeError; end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bugsnag
 version: !ruby/object:Gem::Version
-  version: 6.0.0
+  version: 6.0.1
 platform: ruby
 authors:
 - James Smith
@@ -70,6 +70,7 @@ files:
 - lib/bugsnag/tasks/bugsnag.rake
 - lib/bugsnag/version.rb
 - lib/generators/bugsnag/bugsnag_generator.rb
+- spec/bugsnag_spec.rb
 - spec/cleaner_spec.rb
 - spec/configuration_spec.rb
 - spec/fixtures/crashes/end_of_file.rb
@@ -108,7 +109,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.13
+rubygems_version: 2.4.5
 signing_key: 
 specification_version: 4
 summary: Ruby notifier for bugsnag.com