airbrake-ruby 4.8.0 → 5.2.0

data/spec/thread_pool_spec.rb

@@ -7,7 +7,7 @@ RSpec.describe Airbrake::ThreadPool do
     described_class.new(
       worker_size: worker_size,
       queue_size: queue_size,
-      block: proc { |message| tasks << message }
+      block: proc { |message| tasks << message },
     )
   end
 
@@ -35,7 +35,7 @@ RSpec.describe Airbrake::ThreadPool do
         described_class.new(
           worker_size: 1,
           queue_size: 1,
-          block: proc { |message| tasks << message }
+          block: proc { |message| tasks << message },
         )
       end
 
@@ -51,6 +51,15 @@ RSpec.describe Airbrake::ThreadPool do
 
         expect(tasks.size).to be_zero
       end
+
+      it "logs discarded tasks" do
+        expect(Airbrake::Loggable.instance).to receive(:error).with(
+          /reached its capacity/,
+        ).exactly(15).times
+
+        15.times { subject << 1 }
+        subject.close
+      end
     end
   end
 
@@ -76,11 +85,31 @@ RSpec.describe Airbrake::ThreadPool do
       expect(subject).not_to have_workers
     end
 
-    it "respawns workers on fork()", skip: %w[jruby].include?(RUBY_ENGINE) do
-      pid = fork { expect(subject).to have_workers }
-      Process.wait(pid)
-      subject.close
-      expect(subject).not_to have_workers
+    describe "forking behavior" do
+      before do
+        skip('fork() is unsupported on JRuby') if %w[jruby].include?(RUBY_ENGINE)
+        unless Process.respond_to?(:last_status)
+          skip('Process.last_status is unsupported on this Ruby')
+        end
+      end
+
+      it "respawns workers on fork()" do
+        pid = fork { expect(subject).to have_workers }
+        Process.wait(pid)
+        subject.close
+
+        expect(Process.last_status).to be_success
+        expect(subject).not_to have_workers
+      end
+
+      it "ensures that a new thread group is created per process" do
+        subject << 1
+        pid = fork { subject.has_workers? }
+        Process.wait(pid)
+        subject.close
+
+        expect(Process.last_status).to be_success
+      end
     end
   end
 
@@ -98,11 +127,11 @@ RSpec.describe Airbrake::ThreadPool do
     context "when there's some work to do" do
       it "logs how many tasks are left to process" do
         thread_pool = described_class.new(
-          worker_size: 0, queue_size: 2, block: proc {}
+          worker_size: 0, queue_size: 2, block: proc {},
         )
 
         expect(Airbrake::Loggable.instance).to receive(:debug).with(
-          /waiting to process \d+ task\(s\)/
+          /waiting to process \d+ task\(s\)/,
         )
         expect(Airbrake::Loggable.instance).to receive(:debug).with(/closed/)
 
@@ -112,7 +141,7 @@ RSpec.describe Airbrake::ThreadPool do
 
       it "waits until the queue gets empty" do
         thread_pool = described_class.new(
-          worker_size: 1, queue_size: 2, block: proc {}
+          worker_size: 1, queue_size: 2, block: proc {},
         )
 
         10.times { subject << 1 }

data/spec/timed_trace_spec.rb

@@ -117,7 +117,7 @@ RSpec.describe Airbrake::TimedTrace do
       it "returns a Hash with all spans" do
         expect(subject.spans).to match(
           'operation' => be > 0,
-          'another operation' => be > 0
+          'another operation' => be > 0,
         )
       end
     end

data/spec/truncator_spec.rb

@@ -25,7 +25,7 @@ RSpec.describe Airbrake::Truncator do
           banana: multiply_by_2_max_len('a'),
           kiwi: multiply_by_2_max_len('b'),
           strawberry: 'c',
-          shrimp: 'd'
+          shrimp: 'd',
         }.freeze
       end
 
@@ -34,7 +34,7 @@ RSpec.describe Airbrake::Truncator do
         expect(subject).to be_frozen
 
         expect(subject).to eq(
-          banana: 'aaa[Truncated]', kiwi: 'bbb[Truncated]', strawberry: 'c'
+          banana: 'aaa[Truncated]', kiwi: 'bbb[Truncated]', strawberry: 'c',
         )
         expect(subject[:banana]).to be_frozen
         expect(subject[:kiwi]).to be_frozen
@@ -48,7 +48,7 @@ RSpec.describe Airbrake::Truncator do
           multiply_by_2_max_len('a'),
           'b',
           multiply_by_2_max_len('c'),
-          'd'
+          'd',
         ].freeze
       end
 
@@ -69,7 +69,7 @@ RSpec.describe Airbrake::Truncator do
           multiply_by_2_max_len('a'),
           'b',
           multiply_by_2_max_len('c'),
-          'd'
+          'd',
         ]).freeze
       end
 
@@ -78,7 +78,7 @@ RSpec.describe Airbrake::Truncator do
         expect(subject).to be_frozen
 
         expect(subject).to eq(
-          Set.new(['aaa[Truncated]', 'b', 'ccc[Truncated]'])
+          Set.new(['aaa[Truncated]', 'b', 'ccc[Truncated]']),
         )
         expect(subject).to be_frozen
       end
@@ -166,7 +166,7 @@ RSpec.describe Airbrake::Truncator do
 
       it "prevents recursion" do
         expect(subject).to eq(
-          Set.new(['[Circular]', { k: '[Circular]' }, 'aaa[Truncated]'])
+          Set.new(['[Circular]', { k: '[Circular]' }, 'aaa[Truncated]']),
         )
         expect(subject).to be_frozen
       end
@@ -177,13 +177,13 @@ RSpec.describe Airbrake::Truncator do
         {
           a: multiply_by_2_max_len('a'),
           b: multiply_by_2_max_len('b'),
-          c: { d: multiply_by_2_max_len('d'), e: 'e' }
+          c: { d: multiply_by_2_max_len('d'), e: 'e' },
         }
       end
 
       it "truncates the long strings" do
         expect(subject).to eq(
-          a: 'aaa[Truncated]', b: 'bbb[Truncated]', c: { d: 'ddd[Truncated]', e: 'e' }
+          a: 'aaa[Truncated]', b: 'bbb[Truncated]', c: { d: 'ddd[Truncated]', e: 'e' },
         )
         expect(subject).to be_frozen
       end
@@ -218,8 +218,8 @@ RSpec.describe Airbrake::Truncator do
           errors: [
             { file: 'a' },
             { file: 'a' },
-            hashie.new.merge(file: 'bcde')
-          ]
+            hashie.new.merge(file: 'bcde'),
+          ],
         }
       end
 
@@ -228,8 +228,8 @@ RSpec.describe Airbrake::Truncator do
           errors: [
             { file: 'a' },
             { file: 'a' },
-            hashie.new.merge(file: 'bcd[Truncated]')
-          ]
+            hashie.new.merge(file: 'bcd[Truncated]'),
+          ],
         )
         expect(subject).to be_frozen
       end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: airbrake-ruby
 version: !ruby/object:Gem::Version
-  version: 4.8.0
+  version: 5.2.0
 platform: ruby
 authors:
 - Airbrake Technologies, Inc.
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-10-23 00:00:00.000000000 Z
+date: 2020-12-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rbtree3
@@ -44,6 +44,7 @@ files:
 - lib/airbrake-ruby/benchmark.rb
 - lib/airbrake-ruby/code_hunk.rb
 - lib/airbrake-ruby/config.rb
+- lib/airbrake-ruby/config/processor.rb
 - lib/airbrake-ruby/config/validator.rb
 - lib/airbrake-ruby/deploy_notifier.rb
 - lib/airbrake-ruby/file_cache.rb
@@ -55,17 +56,19 @@ files:
 - lib/airbrake-ruby/filters/git_last_checkout_filter.rb
 - lib/airbrake-ruby/filters/git_repository_filter.rb
 - lib/airbrake-ruby/filters/git_revision_filter.rb
-- lib/airbrake-ruby/filters/keys_blacklist.rb
+- lib/airbrake-ruby/filters/keys_allowlist.rb
+- lib/airbrake-ruby/filters/keys_blocklist.rb
 - lib/airbrake-ruby/filters/keys_filter.rb
-- lib/airbrake-ruby/filters/keys_whitelist.rb
 - lib/airbrake-ruby/filters/root_directory_filter.rb
 - lib/airbrake-ruby/filters/sql_filter.rb
 - lib/airbrake-ruby/filters/system_exit_filter.rb
 - lib/airbrake-ruby/filters/thread_filter.rb
+- lib/airbrake-ruby/grouppable.rb
 - lib/airbrake-ruby/hash_keyable.rb
 - lib/airbrake-ruby/ignorable.rb
 - lib/airbrake-ruby/inspectable.rb
 - lib/airbrake-ruby/loggable.rb
+- lib/airbrake-ruby/mergeable.rb
 - lib/airbrake-ruby/monotonic_time.rb
 - lib/airbrake-ruby/nested_exception.rb
 - lib/airbrake-ruby/notice.rb
@@ -74,6 +77,10 @@ files:
 - lib/airbrake-ruby/performance_notifier.rb
 - lib/airbrake-ruby/promise.rb
 - lib/airbrake-ruby/query.rb
+- lib/airbrake-ruby/queue.rb
+- lib/airbrake-ruby/remote_settings.rb
+- lib/airbrake-ruby/remote_settings/callback.rb
+- lib/airbrake-ruby/remote_settings/settings_data.rb
 - lib/airbrake-ruby/request.rb
 - lib/airbrake-ruby/response.rb
 - lib/airbrake-ruby/stashable.rb
@@ -90,6 +97,7 @@ files:
 - spec/backtrace_spec.rb
 - spec/benchmark_spec.rb
 - spec/code_hunk_spec.rb
+- spec/config/processor_spec.rb
 - spec/config/validator_spec.rb
 - spec/config_spec.rb
 - spec/deploy_notifier_spec.rb
@@ -102,8 +110,8 @@ files:
 - spec/filters/git_last_checkout_filter_spec.rb
 - spec/filters/git_repository_filter.rb
 - spec/filters/git_revision_filter_spec.rb
-- spec/filters/keys_blacklist_spec.rb
-- spec/filters/keys_whitelist_spec.rb
+- spec/filters/keys_allowlist_spec.rb
+- spec/filters/keys_blocklist_spec.rb
 - spec/filters/root_directory_filter_spec.rb
 - spec/filters/sql_filter_spec.rb
 - spec/filters/system_exit_filter_spec.rb
@@ -117,6 +125,7 @@ files:
 - spec/helpers.rb
 - spec/ignorable_spec.rb
 - spec/inspectable_spec.rb
+- spec/loggable_spec.rb
 - spec/monotonic_time_spec.rb
 - spec/nested_exception_spec.rb
 - spec/notice_notifier/options_spec.rb
@@ -126,6 +135,10 @@ files:
 - spec/performance_notifier_spec.rb
 - spec/promise_spec.rb
 - spec/query_spec.rb
+- spec/queue_spec.rb
+- spec/remote_settings/callback_spec.rb
+- spec/remote_settings/settings_data_spec.rb
+- spec/remote_settings_spec.rb
 - spec/request_spec.rb
 - spec/response_spec.rb
 - spec/spec_helper.rb
@@ -141,7 +154,7 @@ homepage: https://airbrake.io
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -149,16 +162,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.1'
+      version: '2.3'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6.2
-signing_key: 
+rubygems_version: 3.1.2
+signing_key:
 specification_version: 4
 summary: Ruby notifier for https://airbrake.io
 test_files:
@@ -168,14 +180,14 @@ test_files:
 - spec/filters/exception_attributes_filter_spec.rb
 - spec/filters/root_directory_filter_spec.rb
 - spec/filters/sql_filter_spec.rb
-- spec/filters/keys_whitelist_spec.rb
 - spec/filters/system_exit_filter_spec.rb
 - spec/filters/thread_filter_spec.rb
+- spec/filters/keys_allowlist_spec.rb
 - spec/filters/dependency_filter_spec.rb
 - spec/filters/context_filter_spec.rb
 - spec/filters/git_last_checkout_filter_spec.rb
 - spec/filters/git_revision_filter_spec.rb
-- spec/filters/keys_blacklist_spec.rb
+- spec/filters/keys_blocklist_spec.rb
 - spec/filters/gem_root_filter_spec.rb
 - spec/filters/git_repository_filter.rb
 - spec/spec_helper.rb
@@ -185,12 +197,14 @@ test_files:
 - spec/tdigest_spec.rb
 - spec/async_sender_spec.rb
 - spec/stat_spec.rb
+- spec/loggable_spec.rb
 - spec/backtrace_spec.rb
 - spec/notice_notifier_spec.rb
 - spec/time_truncate_spec.rb
 - spec/promise_spec.rb
 - spec/thread_pool_spec.rb
 - spec/config/validator_spec.rb
+- spec/config/processor_spec.rb
 - spec/sync_sender_spec.rb
 - spec/ignorable_spec.rb
 - spec/deploy_notifier_spec.rb
@@ -202,7 +216,10 @@ test_files:
 - spec/request_spec.rb
 - spec/notice_notifier/options_spec.rb
 - spec/filter_chain_spec.rb
+- spec/remote_settings/settings_data_spec.rb
+- spec/remote_settings/callback_spec.rb
 - spec/response_spec.rb
+- spec/queue_spec.rb
 - spec/code_hunk_spec.rb
 - spec/fixtures/notroot.txt
 - spec/fixtures/project_root/long_line.txt
@@ -214,3 +231,4 @@ test_files:
 - spec/inspectable_spec.rb
 - spec/stashable_spec.rb
 - spec/query_spec.rb
+- spec/remote_settings_spec.rb