rollbar 0.13.1 → 0.13.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 87940d1312d7031a35a8fffa52ce5c8a3a85028a
-  data.tar.gz: d8cdb363a959a1da8b1262e56ca30b93894dd117
+  metadata.gz: b3f1bfea5b0039c34e065755ca4d589cfc5756b8
+  data.tar.gz: 2ccff055b8ca9a5e3d5e95f5ba57874de697acc8
 SHA512:
-  metadata.gz: a73fb54fe0586c55901fb2164afdc144419a643fb7d8ba05d3f0ec63cb6ac522da1c62ac86d55e223a07d9b4c2f9d89900a8eb62068371544bb92e6a0c5eb621
-  data.tar.gz: a07997b90331f77dbd91c26d81004dcf66231879c433af9aba912a41f5dced783d53c3c3b1910d70e2d5271b909c7938d69be6b91c23417f5577fe3bd456bf56
+  metadata.gz: f6e7fdacbe2ca39f560f4d616a6282413b6d170c8b8aaeca4f616725d84be8beaf55e2318c994aebf1b74028c42824401da745645c553c2671f60867b37be4d0
+  data.tar.gz: a3ba21e9df754a5f47381a5f83f6aa90e4554ab4720af6e543e15d6124450600825f64e9e0271f2c08e6cf87edc9262373aa88e173a3c8b739ad7247526f58df

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Change Log
 
+**0.13.2**
+- Sidekiq payload is no longer mutated when Rollbar reports a Sidekiq job exception
+- Fix sucker_punch async reporting when using a forking application server such as Unicorn (`preload_app true`). Jobs are now instantiated for every report instead of a reused global job instance
+
 **0.13.1**
 - Silence warning when using project_gems= with regexp [#120](https://github.com/rollbar/rollbar-gem/pull/120)
 

data/lib/rollbar/configuration.rb

@@ -87,7 +87,7 @@ module Rollbar
     def use_sucker_punch
       require 'rollbar/delay/sucker_punch' if defined?(SuckerPunch)
       @use_async      = true
-      @async_handler  = Rollbar::Delay::SuckerPunch.new
+      @async_handler  = Rollbar::Delay::SuckerPunch
     end
 
     def use_sucker_punch=(value)

data/lib/rollbar/delay/sucker_punch.rb

@@ -6,8 +6,8 @@ module Rollbar
 
       include ::SuckerPunch::Job
 
-      def call(payload)
-        async.perform payload
+      def self.call(payload)
+        new.async.perform payload
       end
 
       def perform(*args)

data/lib/rollbar/sidekiq.rb

@@ -1,5 +1,7 @@
 # encoding: utf-8
 
+PARAM_BLACKLIST = ['backtrace', 'error_backtrace', 'error_message', 'error_class']
+
 if Sidekiq::VERSION < '3'
   module Rollbar
     class Sidekiq
@@ -7,12 +9,9 @@ if Sidekiq::VERSION < '3'
         begin
           yield
         rescue Exception => e
-          msg.delete('backtrace')
-          msg.delete('error_backtrace')
-          msg.delete('error_message')
-          msg.delete('error_class')
+          params = msg.reject{ |k| PARAM_BLACKLIST.include?(k) }
 
-          Rollbar.report_exception(e, :params => msg)
+          Rollbar.report_exception(e, :params => params)
           raise
         end
       end
@@ -27,12 +26,9 @@ if Sidekiq::VERSION < '3'
 else
   Sidekiq.configure_server do |config|
     config.error_handlers << Proc.new do |e, context|
-      context.delete('backtrace')
-      context.delete('error_backtrace')
-      context.delete('error_message')
-      context.delete('error_class')
-
-      Rollbar.report_exception(e, :params => context)
+      params = context.reject{ |k| PARAM_BLACKLIST.include?(k) }
+      
+      Rollbar.report_exception(e, :params => params)
     end
   end
 end

data/lib/rollbar/version.rb

@@ -1,3 +1,3 @@
 module Rollbar
-  VERSION = "0.13.1"
+  VERSION = "0.13.2"
 end

data/spec/delay/sucker_punch_spec.rb

@@ -13,13 +13,13 @@ rescue LoadError
 end
 
 describe Rollbar::Delay::SuckerPunch, :if => RUBY_VERSION != '1.8.7' do
-  describe "#call" do
+  describe ".call" do
     let(:payload) { "anything" }
 
-    it "performs asynchronously the task" do
+    it "performs the task asynchronously" do
       Rollbar.should_receive(:process_payload)
 
-      subject.call payload
+      Rollbar::Delay::SuckerPunch.call payload
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rollbar
 version: !ruby/object:Gem::Version
-  version: 0.13.1
+  version: 0.13.2
 platform: ruby
 authors:
 - Brian Rue
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-06-30 00:00:00.000000000 Z
+date: 2014-07-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: multi_json