rollbar 0.12.5 → 0.12.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 23c9d47e5d49503a01978ee4d8e4fa00816522de
-  data.tar.gz: fecba94515d4ed5a804d735d73b54dcdb048edd3
+  metadata.gz: 46603900db3bb897dbbdd72baa78f21e8c39ad43
+  data.tar.gz: ba03f938cc791c7db382f879e40ace48c582fd51
 SHA512:
-  metadata.gz: 5cbab0b59a0cc48c1798c976d3891068a8ce43ed5c3b2419323adb7cd11f79fadf14577448f86890f08a5358284ed123211f585e4c08c3af95a66e835414db17
-  data.tar.gz: 022d97884a1ff339520f7492ce22fedc77536a710588b47ba2cf37a75e6646fb08d515a2edfeee5eac94040c6afd1a8c5fb3e360d14f2e8d215f1f3228414e74
+  metadata.gz: 44e60e68802ca3cd885b8ae927c533d187dd213cac504843fe5edb4430778e26595306a24807f4263ca5798050424cbd7244346f201c61570fef67625d2919a9
+  data.tar.gz: 34d2198535bf88d45ed9df35e9c3442427ed5e63b5686c5cc27a764887f76f1918609c23fa8a04d3fa45759c47ed9dca166f3c82944ffb7e93cf7a32ec669319

data/CHANGELOG.md

@@ -1,5 +1,8 @@
 # Change Log
 
+**0.12.6**
+- Added [#78](https://github.com/rollbar/rollbar-gem/pull/78), ability to ignore exceptions from specific persons
+
 **0.12.5**
 - Fixed SIGSEGV with the delayed_job plugin and Ruby 2.1.0
 

data/THANKS.md

@@ -17,6 +17,7 @@ Huge thanks to the following contributors (by github username). For the most up-
 - [juggler](https://github.com/juggler)
 - [kavu](https://github.com/kavu)
 - [magnolia-fan](https://github.com/magnolia-fan)
+- [miloops](https://github.com/miloops)
 - [mipearson](https://github.com/mipearson)
 - [rogercampos](https://github.com/rogercampos)
 - [siong1987](https://github.com/siong1987)

data/lib/rollbar.rb

@@ -20,7 +20,7 @@ require 'rollbar/railtie' if defined?(Rails)
 module Rollbar
   class << self
     attr_writer :configuration
-    attr_reader :last_report
+    attr_accessor :last_report
 
     # Configures the gem.
     #
@@ -74,6 +74,11 @@ module Rollbar
     # @param person_data [Hash] Data describing the affected person. Should be the result of calling
     #   `rollbar_person_data`
     def report_exception(exception, request_data = nil, person_data = nil, level = nil)
+      if person_data.present?
+        person_id = person_data[Rollbar.configuration.person_id_method.to_sym]
+        return 'ignored' if configuration.ignored_person_ids.include?(person_id)
+      end
+      
       return 'disabled' unless configuration.enabled
       return 'ignored' if ignored?(exception)
 
@@ -345,12 +350,12 @@ module Rollbar
 
     def base_data(level = 'error')
       config = configuration
-      
+
       environment = config.environment
       if environment.nil? || environment.empty?
         environment = 'unspecified'
       end
-      
+
       data = {
         :timestamp => Time.now.to_i,
         :environment => environment,

data/lib/rollbar/configuration.rb

@@ -15,6 +15,7 @@ module Rollbar
     attr_accessor :exception_level_filters
     attr_accessor :filepath
     attr_accessor :framework
+    attr_accessor :ignored_person_ids
     attr_accessor :logger
     attr_accessor :person_method
     attr_accessor :person_id_method
@@ -46,6 +47,7 @@ module Rollbar
         'ActionController::RoutingError' => 'warning'
       }
       @framework = 'Plain'
+      @ignored_person_ids = []
       @person_method = 'current_user'
       @person_id_method = 'id'
       @person_username_method = 'username'

data/lib/rollbar/version.rb

@@ -1,3 +1,3 @@
 module Rollbar
-  VERSION = "0.12.5"
+  VERSION = "0.12.6"
 end

data/spec/dummyapp/app/models/user.rb

@@ -1,4 +1,3 @@
 class User < ActiveRecord::Base
-  # Setup accessible (or protected) attributes for your model
   attr_accessible :username, :email, :password, :password_confirmation, :remember_me
 end

data/spec/rollbar_spec.rb

@@ -127,6 +127,47 @@ describe Rollbar do
       end
     end
 
+    it 'should ignore ignored persons' do
+      Rollbar.configure do |config|
+        config.ignored_person_ids += [1]
+      end
+
+      logger_mock.should_not_receive(:info)
+      logger_mock.should_not_receive(:warn)
+      logger_mock.should_not_receive(:error)
+
+      person_data = {
+        :id => 1,
+        :username => "test",
+        :email => "test@example.com"
+      }
+      Rollbar.report_exception(@exception, {}, person_data)
+    end
+    
+    it 'should not ignore non-ignored persons' do
+      Rollbar.configure do |config|
+        config.ignored_person_ids += [1]
+      end
+      
+      Rollbar.last_report = nil
+      
+      person_data = {
+        :id => 1,
+        :username => "test",
+        :email => "test@example.com"
+      }
+      Rollbar.report_exception(@exception, {}, person_data)
+      Rollbar.last_report.should be_nil
+      
+      person_data = {
+        :id => 2,
+        :username => "test2",
+        :email => "test2@example.com"
+      }
+      Rollbar.report_exception(@exception, {}, person_data)
+      Rollbar.last_report.should_not be_nil
+    end
+
     it 'should allow callables to set exception filtered level' do
       callable_mock = double
       saved_filters = Rollbar.configuration.exception_level_filters
@@ -692,7 +733,7 @@ describe Rollbar do
       @dummy_class = DummyClass.new
       @dummy_class.extend(Rollbar::RequestDataExtractor)
     end
-    
+
     context "rollbar_headers" do
       it "should not include cookies" do
         env = {"HTTP_USER_AGENT" => "test", "HTTP_COOKIE" => "cookie"}

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rollbar
 version: !ruby/object:Gem::Version
-  version: 0.12.5
+  version: 0.12.6
 platform: ruby
 authors:
 - Brian Rue