Merge pull request #502 from bugsnag/v6.10.0

v6.10.0 Release

CHANGELOG.md

@@ -1,6 +1,22 @@
 Changelog
 =========
 
+## 6.10.0 (05 Dec 2018)
+
+### Enhancements
+
+* Add SignalException to our default ignored classes
+  | [#479](https://github.com/bugsnag/bugsnag-ruby/pull/479)
+  | [Toby Hsieh](https://github.com/tobyhs)
+
+* Include Bugsnag frames, marked out of project
+  | [#497](https://github.com/bugsnag/bugsnag-ruby/pull/497)
+
+### Fixes
+
+* Ensure Sidekiq request data is always attached to notifications
+  | [#495](https://github.com/bugsnag/bugsnag-ruby/pull/495)
+
 ## 6.9.0 (12 Nov 2018)
 
 ### Enhancements

VERSION

@@ -1 +1 @@
-6.9.0
+6.10.0

lib/bugsnag/configuration.rb

@@ -69,9 +69,9 @@ def initialize
       self.auto_capture_sessions = false
       self.session_endpoint = DEFAULT_SESSION_ENDPOINT
 
-      # SystemExit and Interrupt are common Exception types seen with successful
-      # exits and are not automatically reported to Bugsnag
-      self.ignore_classes = Set.new([SystemExit, Interrupt])
+      # SystemExit and SignalException are common Exception types seen with
+      # successful exits and are not automatically reported to Bugsnag
+      self.ignore_classes = Set.new([SystemExit, SignalException])
 
       # Read the API key from the environment
       self.api_key = ENV["BUGSNAG_API_KEY"]

lib/bugsnag/integrations/mailman.rb

@@ -21,7 +21,6 @@ def call(mail)
         Bugsnag.configuration.set_request_data :mailman_msg, mail.to_s
         yield
       rescue Exception => ex
-        raise ex if [Interrupt, SystemExit, SignalException].include? ex.class
         Bugsnag.notify(ex, true) do |report|
           report.severity = "error"
           report.severity_reason = {

lib/bugsnag/integrations/shoryuken.rb

@@ -27,14 +27,12 @@ def call(_, queue, _, body)
 
         yield
       rescue Exception => ex
-        unless [Interrupt, SystemExit, SignalException].include?(ex.class)
-          Bugsnag.notify(ex, true) do |report|
-            report.severity = "error"
-            report.severity_reason = {
-              :type => Bugsnag::Report::UNHANDLED_EXCEPTION_MIDDLEWARE,
-              :attributes => Bugsnag::Shoryuken::FRAMEWORK_ATTRIBUTES
-            }
-          end
+        Bugsnag.notify(ex, true) do |report|
+          report.severity = "error"
+          report.severity_reason = {
+            :type => Bugsnag::Report::UNHANDLED_EXCEPTION_MIDDLEWARE,
+            :attributes => Bugsnag::Shoryuken::FRAMEWORK_ATTRIBUTES
+          }
         end
         raise
       ensure

lib/bugsnag/integrations/sidekiq.rb

@@ -34,7 +34,6 @@ def call(worker, msg, queue)
     end
 
     def self.notify(exception)
-      return if [Interrupt, SystemExit, SignalException].include? exception.class
       Bugsnag.notify(exception, true) do |report|
         report.severity = "error"
         report.severity_reason = {

spec/configuration_spec.rb

@@ -219,7 +219,7 @@ def debug(name, &block)
   end
 
   it "should have exit exception classes ignored by default" do
-    expect(subject.ignore_classes).to eq(Set.new([SystemExit, Interrupt]))
+    expect(subject.ignore_classes).to eq(Set.new([SystemExit, SignalException]))
   end
 
 end

spec/report_spec.rb

@@ -146,18 +146,27 @@ def gloops
   end
 
   it "sets correct severity and reason for specific error classes" do
-    Bugsnag.notify(SignalException.new("TERM"))
-    expect(Bugsnag).to have_sent_notification{ |payload, headers|
-      event = get_event_from_payload(payload)
-      expect(event["unhandled"]).to be false
-      expect(event["severity"]).to eq("info")
-      expect(event["severityReason"]).to eq({
-        "type" => "errorClass",
-        "attributes" => {
-          "errorClass" => "SignalException"
-        }
-      })
-    }
+    original_ignore_classes = Bugsnag.configuration.ignore_classes
+
+    begin
+      # The default ignore_classes includes SignalException, so we need to
+      # temporarily set it to something else.
+      Bugsnag.configuration.ignore_classes = Set[SystemExit]
+      Bugsnag.notify(SignalException.new("TERM"))
+      expect(Bugsnag).to have_sent_notification{ |payload, headers|
+        event = get_event_from_payload(payload)
+        expect(event["unhandled"]).to be false
+        expect(event["severity"]).to eq("info")
+        expect(event["severityReason"]).to eq({
+          "type" => "errorClass",
+          "attributes" => {
+            "errorClass" => "SignalException"
+          }
+        })
+      }
+    ensure
+      Bugsnag.configuration.ignore_classes = original_ignore_classes
+    end
   end
 
   # TODO: nested context
@@ -1062,7 +1071,7 @@ def gloops
       expect(exception["message"]).to eq("'nil' was notified as an exception")
     }
   end
-
+  
   it "includes bugsnag lines marked out of project" do
     notify_test_exception
     expect(Bugsnag).to have_sent_notification{ |payload, headers|