rollbar 0.10.9 → 0.10.10

data/CHANGELOG.md

@@ -1,5 +1,8 @@
 # Change Log
 
+**0.10.10**
+- Add support for sucker_punch asynchronous handling
+
 **0.10.9**
 - Fix regression introduced in 0.10.7 when ActiveRecord is not present.
 

data/Gemfile

@@ -5,5 +5,4 @@ gemspec
 gem 'sqlite3',                          :platform => [:ruby, :mswin, :mingw]
 gem 'jruby-openssl',                    :platform => :jruby
 gem 'activerecord-jdbcsqlite3-adapter', :platform => :jruby
-gem 'girl_friday'
 gem 'appraisal'

data/THANKS.md

@@ -5,6 +5,7 @@ Huge thanks to the following contributors (by github username). For the most up-
 - [arr-ee](https://github.com/arr-ee)
 - [awmichel](https://github.com/awmichel)
 - [dimko](https://github.com/dimko)
+- [dlackty](https://github.com/dlackty)
 - [fabsays](https://github.com/fabsays)
 - [firstbanco](https://github.com/firstbanco)
 - [ixti](https://github.com/ixti)

data/gemfiles/rails30.gemfile

@@ -5,7 +5,6 @@ source "https://rubygems.org"
 gem "sqlite3", :platform=>[:ruby, :mswin, :mingw]
 gem "jruby-openssl", :platform=>:jruby
 gem "activerecord-jdbcsqlite3-adapter", :platform=>:jruby
-gem "girl_friday"
 gem "appraisal"
 gem "rails", "3.0.20"
 

data/gemfiles/rails31.gemfile

@@ -5,7 +5,6 @@ source "https://rubygems.org"
 gem "sqlite3", :platform=>[:ruby, :mswin, :mingw]
 gem "jruby-openssl", :platform=>:jruby
 gem "activerecord-jdbcsqlite3-adapter", :platform=>:jruby
-gem "girl_friday"
 gem "appraisal"
 gem "rails", "3.1.12"
 

data/gemfiles/rails32.gemfile

@@ -5,7 +5,6 @@ source "https://rubygems.org"
 gem "sqlite3", :platform=>[:ruby, :mswin, :mingw]
 gem "jruby-openssl", :platform=>:jruby
 gem "activerecord-jdbcsqlite3-adapter", :platform=>:jruby
-gem "girl_friday"
 gem "appraisal"
 gem "rails", "3.2.12"
 

data/gemfiles/rails40.gemfile

@@ -5,7 +5,6 @@ source "https://rubygems.org"
 gem "sqlite3", :platform=>[:ruby, :mswin, :mingw]
 gem "jruby-openssl", :platform=>:jruby
 gem "activerecord-jdbcsqlite3-adapter", :platform=>:jruby
-gem "girl_friday"
 gem "appraisal"
 gem "rails", "4.0.0"
 gem "protected_attributes"

data/lib/rollbar.rb

@@ -6,6 +6,7 @@ require 'thread'
 require 'uri'
 
 require 'girl_friday' if defined?(GirlFriday)
+require 'sucker_punch' if defined?(SuckerPunch)
 require 'multi_json'
 
 require 'rollbar/version'

data/lib/rollbar/configuration.rb

@@ -22,11 +22,12 @@ module Rollbar
     attr_accessor :root
     attr_accessor :scrub_fields
     attr_accessor :use_sidekiq
+    attr_accessor :use_sucker_punch
     attr_accessor :use_async
     attr_accessor :use_eventmachine
     attr_accessor :web_base
     attr_accessor :write_to_file
-    
+
     attr_reader :project_gem_paths
 
     DEFAULT_ENDPOINT = 'https://api.rollbar.com/api/1/item/'
@@ -54,11 +55,12 @@ module Rollbar
                        :confirm_password, :password_confirmation, :secret_token]
       @use_async = false
       @use_sidekiq = false
+      @use_sucker_punch = false
       @use_eventmachine = false
       @web_base = DEFAULT_WEB_BASE
       @write_to_file = false
     end
-    
+
     def use_sidekiq=(value)
       if value
         require 'rollbar/delay/sidekiq' if defined?(Sidekiq)
@@ -67,19 +69,27 @@ module Rollbar
       end
     end
 
+    def use_sucker_punch=(value)
+      if value
+        require 'rollbar/delay/sucker_punch' if defined?(SuckerPunch)
+        @use_async      = true
+        @async_handler  = Rollbar::Delay::SuckerPunch.method(:handle)
+      end
+    end
+
     def use_eventmachine=(value)
       require 'em-http-request' if value
       @use_eventmachine = value
     end
-    
+
     def project_gems=(gems)
       @project_gem_paths = []
-      
+
       gems.each { |name|
         begin
           spec = Gem::Specification.find_by_name(name.to_s)
           gem_root = spec.gem_dir
-          
+
           @project_gem_paths.push gem_root
         rescue Gem::LoadError
           puts "[Rollbar] #{name} gem not found"

data/lib/rollbar/delay/sucker_punch.rb

@@ -0,0 +1,18 @@
+require 'sucker_punch'
+
+module Rollbar
+  module Delay
+    class SuckerPunch
+      def self.handle(payload)
+        @@sucker_punch_worker ||= self.new
+        @@sucker_punch_worker.async.perform payload
+      end
+
+      include ::SuckerPunch::Job
+
+      def perform(*args)
+        Rollbar.process_payload(*args)
+      end
+    end
+  end
+end

data/lib/rollbar/rake_tasks.rb

@@ -54,7 +54,10 @@ namespace :rollbar do
 
     puts "Processing..."
     env = Rack::MockRequest.env_for("/verify")
-
-    Rails.application.call(env)
+    status, headers, response = Rails.application.call(env)
+    
+    unless status == 500
+      puts "Test failed! You may have a configuration issue, or you could be using a gem that's blocking the test. Contact support@rollbar.com if you need help troubleshooting."
+    end
   end
 end

data/lib/rollbar/version.rb

@@ -1,3 +1,3 @@
 module Rollbar
-  VERSION = "0.10.9"
+  VERSION = "0.10.10"
 end

data/rollbar.gemspec

@@ -21,5 +21,6 @@ Gem::Specification.new do |gem|
   gem.add_development_dependency 'rspec-rails', '~> 2.12.0'
   gem.add_development_dependency 'database_cleaner', '~> 1.0.0'
   gem.add_development_dependency 'girl_friday', '>= 0.11.1'
+  gem.add_development_dependency 'sucker_punch', '>= 1.0.0' if RUBY_VERSION != '1.8.7'
   gem.add_development_dependency 'genspec', '>= 0.2.7'
 end

data/spec/rollbar_spec.rb

@@ -1,6 +1,13 @@
 require 'logger'
 require 'socket'
 require 'spec_helper'
+require 'girl_friday'
+
+begin
+  require 'sucker_punch'
+  require 'sucker_punch/testing/inline'
+rescue LoadError
+end
 
 describe Rollbar do
 
@@ -170,7 +177,7 @@ describe Rollbar do
       exception = CustomException.new("oops")
 
       Rollbar.report_exception(exception)
-      
+
       payload["data"]["body"]["trace"]["frames"][0]["method"].should == "custom backtrace line"
     end
   end
@@ -344,6 +351,25 @@ describe Rollbar do
       end
     end
 
+    if defined?(SuckerPunch)
+      it "should send the payload to sucker_punch delayer" do
+        logger_mock.should_receive(:info).with('[Rollbar] Scheduling payload')
+        logger_mock.should_receive(:info).with('[Rollbar] Sending payload')
+        logger_mock.should_receive(:info).with('[Rollbar] Success')
+  
+        Rollbar.configure do |config|
+          config.use_sucker_punch = true
+        end
+  
+        Rollbar.report_exception(@exception)
+  
+        Rollbar.configure do |config|
+          config.use_async = false
+          config.async_handler = Rollbar.method(:default_async_handler)
+        end
+      end
+    end
+
     it "should send the payload to sidekiq delayer" do
       module Rollbar
         module Delay

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rollbar
 version: !ruby/object:Gem::Version
-  version: 0.10.9
+  version: 0.10.10
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-08-07 00:00:00.000000000 Z
+date: 2013-08-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: multi_json
@@ -91,6 +91,22 @@ dependencies:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: 0.11.1
+- !ruby/object:Gem::Dependency
+  name: sucker_punch
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: 1.0.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: 1.0.0
 - !ruby/object:Gem::Dependency
   name: genspec
   requirement: !ruby/object:Gem::Requirement
@@ -136,6 +152,7 @@ files:
 - lib/rollbar/capistrano.rb
 - lib/rollbar/configuration.rb
 - lib/rollbar/delay/sidekiq.rb
+- lib/rollbar/delay/sucker_punch.rb
 - lib/rollbar/delayed_job.rb
 - lib/rollbar/exception_reporter.rb
 - lib/rollbar/goalie.rb