airbrake-ruby 6.0.2 → 6.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f0310134480c9c13f2e8ab87a4c616768c213b590f98edd38e9c9abbd539a7ba
-  data.tar.gz: 6f5e8d2411eee4e18ff97ef7d89b3c4a1c8ebb3ba5ec14fafe24c25e850672b1
+  metadata.gz: 2dfccc9b2cf013d06ad40e4b3726319c13bd09dcba70d9c1fa4b382c7d80b1b4
+  data.tar.gz: 63c8aa81c33948f26ead281e81769a274dbca9740880a420d62b4d951b352984
 SHA512:
-  metadata.gz: d6e19e37eca3e38af441560c044ac1472ab06a282c4d8b4873a4c8529b9fb9e9a70634a42b4d0d5fe668ba866649ee115942678e5bbfffa1f60373ccd66e964b
-  data.tar.gz: f8b22c23d2534b1f4dc12dd1b5d8bf970504b297a61421cda6d496ec2f1143ac9eb031e2c1b033e8d8a24b94b634c2a17ea475f9ef13d482be08a419a54b1d98
+  metadata.gz: 694a9114eb130976c052748a51e0490b3276bf32fe347bf466e64caf68e26c703941cd131bb43a0b7f3b6c6c9580067feffd5c1d443a46319a09ec6d0d1f6a37
+  data.tar.gz: 59bf1fd79c2467401aabc0c352397d82bae83805cb54ccac40c5ca132dfcfdefa5281a8ba11dc96f67d569405837af61aeb05a22ce9e2f3425d0ecc4a1c69251

data/lib/airbrake-ruby/config/processor.rb

@@ -43,8 +43,14 @@ module Airbrake
       def process_remote_configuration
         return unless @config.remote_config
         return unless @project_id
+
+        # Never poll remote configuration in the test environment.
         return if @config.environment == 'test'
 
+        # If the current environment is ignored, don't try to poll remote
+        # configuration.
+        return if @config.ignore_environments.include?(@config.environment)
+
         RemoteSettings.poll(@project_id, @config.remote_config_host) do |data|
           @poll_callback.call(data)
         end

data/lib/airbrake-ruby/filters/git_repository_filter.rb

@@ -49,7 +49,13 @@ module Airbrake
       def detect_git_version
         return unless which('git')
 
-        Gem::Version.new(`git --version`.split[2])
+        begin
+          Gem::Version.new(`git --version`.split[2])
+        rescue Errno::EAGAIN
+          # Bugfix for the case when the system cannot allocate memory for
+          # a fork() call: https://github.com/airbrake/airbrake-ruby/issues/680
+          nil
+        end
       end
 
       # Cross-platform way to tell if an executable is accessible.

data/lib/airbrake-ruby/version.rb

@@ -3,7 +3,7 @@
 module Airbrake
   # @return [String] the library version
   # @api public
-  AIRBRAKE_RUBY_VERSION = '6.0.2'.freeze
+  AIRBRAKE_RUBY_VERSION = '6.1.0'.freeze
 
   # @return [Hash{Symbol=>String}] the information about the notifier library
   # @since v5.0.0

data/spec/benchmark_spec.rb

@@ -11,13 +11,13 @@ RSpec.describe Airbrake::Benchmark do
     before { benchmark }
 
     context "when called one time" do
-      its(:stop) { is_expected.to eq(true) }
+      its(:stop) { is_expected.to be(true) }
     end
 
     context "when called twice or more" do
       before { benchmark.stop }
 
-      its(:stop) { is_expected.to eq(false) }
+      its(:stop) { is_expected.to be(false) }
     end
   end
 

data/spec/config/processor_spec.rb

@@ -7,7 +7,7 @@ RSpec.describe Airbrake::Config::Processor do
     context "when there ARE blocklist keys" do
       it "adds the blocklist filter" do
         described_class.new(config).process_blocklist(notifier)
-        expect(notifier.has_filter?(Airbrake::Filters::KeysBlocklist)).to eq(true)
+        expect(notifier.has_filter?(Airbrake::Filters::KeysBlocklist)).to be(true)
       end
     end
 
@@ -17,7 +17,7 @@ RSpec.describe Airbrake::Config::Processor do
       it "doesn't add the blocklist filter" do
         described_class.new(config).process_blocklist(notifier)
         expect(notifier.has_filter?(Airbrake::Filters::KeysBlocklist))
-          .to eq(false)
+          .to be(false)
       end
     end
   end
@@ -28,7 +28,7 @@ RSpec.describe Airbrake::Config::Processor do
     context "when there ARE allowlist keys" do
       it "adds the allowlist filter" do
         described_class.new(config).process_allowlist(notifier)
-        expect(notifier.has_filter?(Airbrake::Filters::KeysAllowlist)).to eq(true)
+        expect(notifier.has_filter?(Airbrake::Filters::KeysAllowlist)).to be(true)
       end
     end
 
@@ -38,7 +38,7 @@ RSpec.describe Airbrake::Config::Processor do
       it "doesn't add the allowlist filter" do
         described_class.new(config).process_allowlist(notifier)
         expect(notifier.has_filter?(Airbrake::Filters::KeysAllowlist))
-          .to eq(false)
+          .to be(false)
       end
     end
   end
@@ -68,6 +68,22 @@ RSpec.describe Airbrake::Config::Processor do
       end
     end
 
+    context "when the config sets :ignore_environments and :environment matches" do
+      let(:config) do
+        Airbrake::Config.new(
+          project_id: 123,
+          ignore_environments: %w[dev],
+          environment: 'dev',
+        )
+      end
+
+      it "doesn't set remote settings" do
+        described_class.new(config).process_remote_configuration
+
+        expect(Airbrake::RemoteSettings).not_to have_received(:poll)
+      end
+    end
+
     context "when the config defines a project_id" do
       let(:config) do
         Airbrake::Config.new(project_id: 123, environment: 'not-test')
@@ -98,25 +114,25 @@ RSpec.describe Airbrake::Config::Processor do
       it "adds RootDirectoryFilter" do
         described_class.new(config).add_filters(notifier)
         expect(notifier.has_filter?(Airbrake::Filters::RootDirectoryFilter))
-          .to eq(true)
+          .to be(true)
       end
 
       it "adds GitRevisionFilter" do
         described_class.new(config).add_filters(notifier)
         expect(notifier.has_filter?(Airbrake::Filters::GitRevisionFilter))
-          .to eq(true)
+          .to be(true)
       end
 
       it "adds GitRepositoryFilter" do
         described_class.new(config).add_filters(notifier)
         expect(notifier.has_filter?(Airbrake::Filters::GitRepositoryFilter))
-          .to eq(true)
+          .to be(true)
       end
 
       it "adds GitLastCheckoutFilter" do
         described_class.new(config).add_filters(notifier)
         expect(notifier.has_filter?(Airbrake::Filters::GitLastCheckoutFilter))
-          .to eq(true)
+          .to be(true)
       end
     end
 
@@ -126,25 +142,25 @@ RSpec.describe Airbrake::Config::Processor do
       it "doesn't add RootDirectoryFilter" do
         described_class.new(config).add_filters(notifier)
         expect(notifier.has_filter?(Airbrake::Filters::RootDirectoryFilter))
-          .to eq(false)
+          .to be(false)
       end
 
       it "doesn't add GitRevisionFilter" do
         described_class.new(config).add_filters(notifier)
         expect(notifier.has_filter?(Airbrake::Filters::GitRevisionFilter))
-          .to eq(false)
+          .to be(false)
       end
 
       it "doesn't add GitRepositoryFilter" do
         described_class.new(config).add_filters(notifier)
         expect(notifier.has_filter?(Airbrake::Filters::GitRepositoryFilter))
-          .to eq(false)
+          .to be(false)
       end
 
       it "doesn't add GitLastCheckoutFilter" do
         described_class.new(config).add_filters(notifier)
         expect(notifier.has_filter?(Airbrake::Filters::GitLastCheckoutFilter))
-          .to eq(false)
+          .to be(false)
       end
     end
   end

data/spec/config_spec.rb

@@ -23,12 +23,12 @@ RSpec.describe Airbrake::Config do
   its(:timeout) { is_expected.to be_nil }
   its(:blocklist_keys) { is_expected.to be_empty }
   its(:allowlist_keys) { is_expected.to be_empty }
-  its(:performance_stats) { is_expected.to eq(true) }
+  its(:performance_stats) { is_expected.to be(true) }
   its(:performance_stats_flush_period) { is_expected.to eq(15) }
-  its(:query_stats) { is_expected.to eq(true) }
-  its(:job_stats) { is_expected.to eq(true) }
-  its(:error_notifications) { is_expected.to eq(true) }
-  its(:remote_config) { is_expected.to eq(true) }
+  its(:query_stats) { is_expected.to be(true) }
+  its(:job_stats) { is_expected.to be(true) }
+  its(:error_notifications) { is_expected.to be(true) }
+  its(:remote_config) { is_expected.to be(true) }
 
   its(:remote_config_host) do
     is_expected.to eq('https://notifier-configs.airbrake.io')

data/spec/filter_chain_spec.rb

@@ -101,14 +101,14 @@ RSpec.describe Airbrake::FilterChain do
         klass = Class.new
 
         filter_chain.add_filter(klass.new)
-        expect(filter_chain.includes?(klass)).to eq(true)
+        expect(filter_chain.includes?(klass)).to be(true)
       end
     end
 
     context "when Proc filter class is included in the filter chain" do
       it "returns true" do
         filter_chain.add_filter(proc {})
-        expect(filter_chain.includes?(Proc)).to eq(true)
+        expect(filter_chain.includes?(Proc)).to be(true)
       end
     end
 
@@ -117,7 +117,7 @@ RSpec.describe Airbrake::FilterChain do
         klass = Class.new
 
         filter_chain.add_filter(proc {})
-        expect(filter_chain.includes?(klass)).to eq(false)
+        expect(filter_chain.includes?(klass)).to be(false)
       end
     end
   end

data/spec/filters/{git_repository_filter.rb → git_repository_filter_spec.rb}

@@ -1,9 +1,7 @@
 RSpec.describe Airbrake::Filters::GitRepositoryFilter do
   subject(:git_repository_filter) { described_class.new('.') }
 
-  let(:notice) do
-    Airbrake::Notice.new(Airbrake::Config.new, AirbrakeTestError.new)
-  end
+  let(:notice) { Airbrake::Notice.new(AirbrakeTestError.new) }
 
   describe "#initialize" do
     it "parses standard git version" do
@@ -23,6 +21,14 @@ RSpec.describe Airbrake::Filters::GitRepositoryFilter do
         .to receive(:`).and_return('git version 2.17.2 (Apple Git-113)')
       expect { git_repository_filter }.not_to raise_error
     end
+
+    context "when Errno::EAGAIN is raised when detecting git version" do
+      it "doesn't attach anything to context/repository" do
+        allow_any_instance_of(Kernel).to receive(:`).and_raise(Errno::EAGAIN)
+        git_repository_filter.call(notice)
+        expect(notice[:context][:repository]).to be_nil
+      end
+    end
   end
 
   context "when context/repository is defined" do
@@ -34,7 +40,7 @@ RSpec.describe Airbrake::Filters::GitRepositoryFilter do
   end
 
   context "when .git directory doesn't exist" do
-    git_repository_filter { described_class.new('root/dir') }
+    subject(:git_repository_filter) { described_class.new('root/dir') }
 
     it "doesn't attach anything to context/repository" do
       git_repository_filter.call(notice)
@@ -45,15 +51,20 @@ RSpec.describe Airbrake::Filters::GitRepositoryFilter do
   context "when .git directory exists" do
     it "attaches context/repository" do
       git_repository_filter.call(notice)
-      expect(notice[:context][:repository]).to eq(
-        'ssh://git@github.com/airbrake/airbrake-ruby.git',
+      expect(notice[:context][:repository]).to match(
+        'github.com/airbrake/airbrake-ruby',
       )
     end
   end
 
   context "when git is not in PATH" do
+    let!(:path) { ENV['PATH'] }
+
+    before { ENV['PATH'] = '' }
+
+    after { ENV['PATH'] = path }
+
     it "does not attach context/repository" do
-      ENV['PATH'] = ''
       git_repository_filter.call(notice)
       expect(notice[:context][:repository]).to be_nil
     end

data/spec/remote_settings/callback_spec.rb

@@ -42,13 +42,13 @@ RSpec.describe Airbrake::RemoteSettings::Callback do
         callback = described_class.new(config)
 
         callback.call(data)
-        expect(config.error_notifications).to eq(false)
+        expect(config.error_notifications).to be(false)
 
         callback.call(data)
-        expect(config.error_notifications).to eq(false)
+        expect(config.error_notifications).to be(false)
 
         callback.call(data)
-        expect(config.error_notifications).to eq(false)
+        expect(config.error_notifications).to be(false)
       end
       # rubocop:enable RSpec/MultipleExpectations
     end
@@ -63,12 +63,12 @@ RSpec.describe Airbrake::RemoteSettings::Callback do
         allow(data).to receive(:error_notifications?).and_return(false)
 
         callback.call(data)
-        expect(config.error_notifications).to eq(false)
+        expect(config.error_notifications).to be(false)
 
         allow(data).to receive(:error_notifications?).and_return(true)
 
         callback.call(data)
-        expect(config.error_notifications).to eq(true)
+        expect(config.error_notifications).to be(true)
 
         expect(data).to have_received(:error_notifications?).twice
       end
@@ -86,13 +86,13 @@ RSpec.describe Airbrake::RemoteSettings::Callback do
         callback = described_class.new(config)
 
         callback.call(data)
-        expect(config.performance_stats).to eq(false)
+        expect(config.performance_stats).to be(false)
 
         callback.call(data)
-        expect(config.performance_stats).to eq(false)
+        expect(config.performance_stats).to be(false)
 
         callback.call(data)
-        expect(config.performance_stats).to eq(false)
+        expect(config.performance_stats).to be(false)
       end
       # rubocop:enable RSpec/MultipleExpectations
     end
@@ -107,12 +107,12 @@ RSpec.describe Airbrake::RemoteSettings::Callback do
         allow(data).to receive(:performance_stats?).and_return(false)
 
         callback.call(data)
-        expect(config.performance_stats).to eq(false)
+        expect(config.performance_stats).to be(false)
 
         allow(data).to receive(:performance_stats?).and_return(true)
 
         callback.call(data)
-        expect(config.performance_stats).to eq(true)
+        expect(config.performance_stats).to be(true)
 
         expect(data).to have_received(:performance_stats?).twice
       end

data/spec/remote_settings/settings_data_spec.rb

@@ -123,7 +123,7 @@ RSpec.describe Airbrake::RemoteSettings::SettingsData do
 
         it "returns true" do
           expect(described_class.new(project_id, data).error_notifications?)
-            .to eq(true)
+            .to be(true)
         end
       end
 
@@ -141,7 +141,7 @@ RSpec.describe Airbrake::RemoteSettings::SettingsData do
 
         it "returns false" do
           expect(described_class.new(project_id, data).error_notifications?)
-            .to eq(false)
+            .to be(false)
         end
       end
     end
@@ -153,7 +153,7 @@ RSpec.describe Airbrake::RemoteSettings::SettingsData do
 
       it "returns true" do
         expect(described_class.new(project_id, data).error_notifications?)
-          .to eq(true)
+          .to be(true)
       end
     end
   end
@@ -174,7 +174,7 @@ RSpec.describe Airbrake::RemoteSettings::SettingsData do
 
         it "returns true" do
           expect(described_class.new(project_id, data).performance_stats?)
-            .to eq(true)
+            .to be(true)
         end
       end
 
@@ -192,7 +192,7 @@ RSpec.describe Airbrake::RemoteSettings::SettingsData do
 
         it "returns false" do
           expect(described_class.new(project_id, data).performance_stats?)
-            .to eq(false)
+            .to be(false)
         end
       end
     end
@@ -204,7 +204,7 @@ RSpec.describe Airbrake::RemoteSettings::SettingsData do
 
       it "returns true" do
         expect(described_class.new(project_id, data).performance_stats?)
-          .to eq(true)
+          .to be(true)
       end
     end
   end

data/spec/remote_settings_spec.rb

@@ -77,8 +77,8 @@ RSpec.describe Airbrake::RemoteSettings do
         sleep(0.1)
         remote_settings.stop_polling
 
-        expect(settings.error_notifications?).to eq(true)
-        expect(settings.performance_stats?).to eq(false)
+        expect(settings.error_notifications?).to be(true)
+        expect(settings.performance_stats?).to be(false)
         expect(settings.interval).to eq(1)
       end
       # rubocop:enable RSpec/MultipleExpectations

data/spec/thread_pool_spec.rb

@@ -15,7 +15,7 @@ RSpec.describe Airbrake::ThreadPool do
     it "returns true" do
       retval = thread_pool << 1
       thread_pool.close
-      expect(retval).to eq(true)
+      expect(retval).to be(true)
     end
 
     it "performs work in background" do
@@ -44,7 +44,7 @@ RSpec.describe Airbrake::ThreadPool do
       it "returns false" do
         retval = full_thread_pool << 1
         full_thread_pool.close
-        expect(retval).to eq(false)
+        expect(retval).to be(false)
       end
 
       it "discards tasks" do

data/spec/timed_trace_spec.rb

@@ -22,7 +22,7 @@ RSpec.describe Airbrake::TimedTrace do
   describe "#start_span" do
     context "when called once" do
       it "returns true" do
-        expect(timed_trace.start_span('operation')).to eq(true)
+        expect(timed_trace.start_span('operation')).to be(true)
       end
     end
 
@@ -30,7 +30,7 @@ RSpec.describe Airbrake::TimedTrace do
       before { timed_trace.start_span('operation') }
 
       it "returns false" do
-        expect(timed_trace.start_span('operation')).to eq(false)
+        expect(timed_trace.start_span('operation')).to be(false)
       end
     end
 
@@ -38,7 +38,7 @@ RSpec.describe Airbrake::TimedTrace do
       before { timed_trace.start_span('operation') }
 
       it "returns true" do
-        expect(timed_trace.start_span('another operation')).to eq(true)
+        expect(timed_trace.start_span('another operation')).to be(true)
       end
     end
 
@@ -54,7 +54,7 @@ RSpec.describe Airbrake::TimedTrace do
   describe "#stop_span" do
     context "when #start_span wasn't invoked" do
       it "returns false" do
-        expect(timed_trace.stop_span('operation')).to eq(false)
+        expect(timed_trace.stop_span('operation')).to be(false)
       end
     end
 
@@ -62,7 +62,7 @@ RSpec.describe Airbrake::TimedTrace do
       before { timed_trace.start_span('operation') }
 
       it "returns true" do
-        expect(timed_trace.stop_span('operation')).to eq(true)
+        expect(timed_trace.stop_span('operation')).to be(true)
       end
     end
 
@@ -74,15 +74,15 @@ RSpec.describe Airbrake::TimedTrace do
 
       context "and when stopping in LIFO order" do
         it "returns true for all spans" do
-          expect(timed_trace.stop_span('another operation')).to eq(true)
-          expect(timed_trace.stop_span('operation')).to eq(true)
+          expect(timed_trace.stop_span('another operation')).to be(true)
+          expect(timed_trace.stop_span('operation')).to be(true)
         end
       end
 
       context "and when stopping in FIFO order" do
         it "returns true for all spans" do
-          expect(timed_trace.stop_span('operation')).to eq(true)
-          expect(timed_trace.stop_span('another operation')).to eq(true)
+          expect(timed_trace.stop_span('operation')).to be(true)
+          expect(timed_trace.stop_span('another operation')).to be(true)
         end
       end
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: airbrake-ruby
 version: !ruby/object:Gem::Version
-  version: 6.0.2
+  version: 6.1.0
 platform: ruby
 authors:
 - Airbrake Technologies, Inc.
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-01-10 00:00:00.000000000 Z
+date: 2022-04-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rbtree3
@@ -110,7 +110,7 @@ files:
 - spec/filters/exception_attributes_filter_spec.rb
 - spec/filters/gem_root_filter_spec.rb
 - spec/filters/git_last_checkout_filter_spec.rb
-- spec/filters/git_repository_filter.rb
+- spec/filters/git_repository_filter_spec.rb
 - spec/filters/git_revision_filter_spec.rb
 - spec/filters/keys_allowlist_spec.rb
 - spec/filters/keys_blocklist_spec.rb
@@ -172,7 +172,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.15
+rubygems_version: 3.2.32
 signing_key:
 specification_version: 4
 summary: Ruby notifier for https://airbrake.io
@@ -194,7 +194,7 @@ test_files:
 - spec/filters/exception_attributes_filter_spec.rb
 - spec/filters/gem_root_filter_spec.rb
 - spec/filters/git_last_checkout_filter_spec.rb
-- spec/filters/git_repository_filter.rb
+- spec/filters/git_repository_filter_spec.rb
 - spec/filters/git_revision_filter_spec.rb
 - spec/filters/keys_allowlist_spec.rb
 - spec/filters/keys_blocklist_spec.rb