bugsnag 6.7.1 → 6.7.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 313641e103c52e05c89cc8d3277a0a7285504523
-  data.tar.gz: 3bbe2d377ff467cec0a868510c507f467385d069
+  metadata.gz: 4a0ddb8da5fb920f1f434e58e955d26b979debe8
+  data.tar.gz: 65de67a4ae0dbb7356bd893100e7889d608cb8bb
 SHA512:
-  metadata.gz: ca1371c07f154c2c3d2241e868b9bb49fefc86cc682f4744d819552dd75d517bdeb04dbb3fb5b11f007efa9fd08458e2d6cd198da810e089381965744206a8d6
-  data.tar.gz: 39beb55e75be93fba3921dec840ed1d118de8b9818df62bd3b4a3a3a6d1bce190b087acb7aa36493986264cd7135020e5e38980fe535e225c9b89df679def20a
+  metadata.gz: 83327beda70eccb521332bdac2fd0c6b96eff182c508bdfacf67ba5465ceac51bb5c3f5df26ab5294c8c6051efc721eda05d9862d561de0a5b1bfe0184b746ad
+  data.tar.gz: cb10e69935a1682cba5aedcceee811097626a95a96a394bcbab0f22aaa77d427fd8ec00d1788a210b0c3354d4b1a8ea567066758fc3df01c1b909496a42a5fb1

data/CHANGELOG.md

@@ -1,6 +1,13 @@
 Changelog
 =========
 
+## 6.7.2 (24 Apr 2018)
+
+### Fixes
+
+* (Notify) Handle `notify` calls with `nil` arguments correctly
+  | [#439](https://github.com/bugsnag/bugsnag-ruby/pull/439)
+
 ## 6.7.1 (11 Apr 2018)
 
 ### Fixes

data/VERSION

@@ -1 +1 @@
-6.7.1
+6.7.2

data/bugsnag.gemspec

@@ -7,7 +7,7 @@ Gem::Specification.new do |s|
 
   s.description = "Ruby notifier for bugsnag.com"
   s.summary = "Ruby notifier for bugsnag.com"
-  s.homepage = "http://github.com/bugsnag/bugsnag-ruby"
+  s.homepage = "https://github.com/bugsnag/bugsnag-ruby"
   s.licenses = ["MIT"]
 
   s.files = `git ls-files`.split("\n").reject {|file| file.start_with? "example/"}

data/issue_template.md

@@ -1,18 +1,48 @@
-### Expected behavior
-*[Insert details on expected behaviour]*
+### Description
+<!-- A quick description of what you're trying to accomplish -->
 
-### Observed behavior
-*[Insert details on observed behaviour]*
+### Issue
+<!--
+  What went wrong? (If this issue is a general question or a proposed change,
+  feel free to delete this and subsequent sections.
+-->
 
-### Steps to reproduce
-*[Insert reproduction steps (if known)]*
+### Environment
 
-### Version
-*[Insert version information]*
+Library versions:
 
-### Additional information
-*[Insert any additional information]*
+<!--
+  Paste the output of this command into the code block below:
+    bundle list | grep -E "(bugsnag|rails|sidekiq|que|sinatra|resque|shoryuken|mailman|delayed_job)"
+-->
+```
 
-#### Can't comment on Issues?
-Some users have been unable to comment on Github issues when an [adblocker extension is enabled](https://docs.bugsnag.com/platforms/browsers/faq/#is-bugsnag-blocked-by-ad-blockers).
-We recommend temporarily disabling the extension, or if that fails, contacting support@bugsnag.com.
+```
+
+- OS / OS version:
+- debug mode or production?:
+
+<!--
+  Below are a few approaches you might take to communicate the issue, in
+  descending order of awesomeness. Please choose one and feel free to delete
+  the others from this template.
+-->
+### Example Repo
+
+- [ ] Create a minimal repository that can reproduce the issue after running
+  `bundle install` and `rackup` (or `rails server`, or `ruby some-script.rb`,
+  etc)
+- [ ] Link to it here:
+
+### Example code snippet
+
+```ruby
+require 'bugsnag'
+
+# (Insert code sample to reproduce the problem)
+```
+
+<!-- Error messages, if any -->
+```
+
+```

data/lib/bugsnag.rb

@@ -32,6 +32,8 @@ module Bugsnag
   LOCK = Mutex.new
   INTEGRATIONS = [:resque, :sidekiq, :mailman, :delayed_job, :shoryuken, :que]
 
+  NIL_EXCEPTION_DESCRIPTION = "'nil' was notified as an exception"
+
   class << self
     ##
     # Configure the Bugsnag notifier application-wide settings.
@@ -53,25 +55,9 @@ module Bugsnag
         auto_notify = false
       end
 
-      if !configuration.auto_notify && auto_notify
-        configuration.debug("Not notifying because auto_notify is disabled")
-        return
-      end
-
-      if !configuration.valid_api_key?
-        configuration.debug("Not notifying due to an invalid api_key")
-        return
-      end
+      return unless deliver_notification?(exception, auto_notify)
 
-      if !configuration.should_notify_release_stage?
-        configuration.debug("Not notifying due to notify_release_stages :#{configuration.notify_release_stages.inspect}")
-        return
-      end
-
-      if exception.respond_to?(:skip_bugsnag) && exception.skip_bugsnag
-        configuration.debug("Not notifying due to skip_bugsnag flag")
-        return
-      end
+      exception = NIL_EXCEPTION_DESCRIPTION if exception.nil?
 
       report = Report.new(exception, configuration, auto_notify)
 
@@ -176,6 +162,26 @@ module Bugsnag
       end
     end
 
+    private
+
+    def deliver_notification?(exception, auto_notify)
+      reason = abort_reason(exception, auto_notify)
+      configuration.debug(reason) unless reason.nil?
+      reason.nil?
+    end
+
+    def abort_reason(exception, auto_notify)
+      if !configuration.auto_notify && auto_notify
+        "Not notifying because auto_notify is disabled"
+      elsif !configuration.valid_api_key?
+        "Not notifying due to an invalid api_key"
+      elsif !configuration.should_notify_release_stage?
+        "Not notifying due to notify_release_stages :#{configuration.notify_release_stages.inspect}"
+      elsif exception.respond_to?(:skip_bugsnag) && exception.skip_bugsnag
+        "Not notifying due to skip_bugsnag flag"
+      end
+    end
+
     # Check if the API key is valid and warn (once) if it is not
     def check_key_valid
       @key_warning = false unless defined?(@key_warning)

data/lib/bugsnag/report.rb

@@ -6,7 +6,7 @@ module Bugsnag
   class Report
     NOTIFIER_NAME = "Ruby Bugsnag Notifier"
     NOTIFIER_VERSION = Bugsnag::VERSION
-    NOTIFIER_URL = "http://www.bugsnag.com"
+    NOTIFIER_URL = "https://www.bugsnag.com"
 
     UNHANDLED_EXCEPTION = "unhandledException"
     UNHANDLED_EXCEPTION_MIDDLEWARE = "unhandledExceptionMiddleware"

data/spec/report_spec.rb

@@ -1045,6 +1045,21 @@ describe Bugsnag::Report do
     expect(Bugsnag).not_to have_sent_notification
   end
 
+  it 'uses an appropriate message if nil is notified' do
+    Bugsnag.notify(nil)
+    expect(Bugsnag).to have_sent_notification{ |payload, headers|
+      event = payload["events"][0]
+      exception = event["exceptions"][0]
+      expect(exception["errorClass"]).to eq("RuntimeError")
+      expect(exception["message"]).to eq("'nil' was notified as an exception")
+
+      stacktrace = exception["stacktrace"][0]
+      expect(stacktrace["lineNumber"]).to eq(1049)
+      expect(stacktrace["file"]).to end_with("spec/report_spec.rb")
+      expect(stacktrace["code"]["1048"]).to eq("  it 'uses an appropriate message if nil is notified' do")
+      expect(stacktrace["code"]["1049"]).to eq("    Bugsnag.notify(nil)")
+    }
+  end
 
   if defined?(JRUBY_VERSION)
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bugsnag
 version: !ruby/object:Gem::Version
-  version: 6.7.1
+  version: 6.7.2
 platform: ruby
 authors:
 - James Smith
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-04-11 00:00:00.000000000 Z
+date: 2018-04-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: concurrent-ruby
@@ -131,7 +131,7 @@ files:
 - spec/session_tracker_spec.rb
 - spec/spec_helper.rb
 - spec/stacktrace_spec.rb
-homepage: http://github.com/bugsnag/bugsnag-ruby
+homepage: https://github.com/bugsnag/bugsnag-ruby
 licenses:
 - MIT
 metadata: {}
@@ -151,7 +151,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.14
+rubygems_version: 2.6.11
 signing_key: 
 specification_version: 4
 summary: Ruby notifier for bugsnag.com