airbrake-ruby 3.2.3 → 3.2.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 540800f5ffc99b82886e6a5055526369ae337bdd
-  data.tar.gz: 984d90b0ae217f25ecfb05ac23dabb5153a306dc
+  metadata.gz: f0151a765b95213dc4a221547aa4afd2cf7da6c0
+  data.tar.gz: b13483675acade75535356f679099873ed234888
 SHA512:
-  metadata.gz: 0b3cebc4c58fac5589425fde6bf4c74e0ded6dc4d19ce89ddc38463e9a569fbf30ece2215c0eb071b4ba08109c02c4cf1d0f707b01eeca4b4279946b29820731
-  data.tar.gz: b988834c61b2562b94e39ccea3b0443ac42faadcfae129d40ec33867edf184f6fd3fa02b332dc7693fb57c5c657002dc4d136b1a4f996d464d458b3d47c13a60
+  metadata.gz: fe9e46d9281e883be340f1648160989c4e8683c52e7dfec8652ba0403d625d70d2d23874134c2fc6000b0a3d8730dfeb01778832e1cd763999278883ad13c115
+  data.tar.gz: 39a05948f1e8176e7862cda8a02d669a9f2f46e078f1aca1901167814192e1ea1ad62f6b0b81df802f8a2b5686fc6ac6f0295b52daa2a121ddaf8d48eb79fc42

data/lib/airbrake-ruby/config.rb

@@ -206,7 +206,7 @@ module Airbrake
     def route_stats=(value)
       logger.warn(
         "#{LOG_LABEL} the 'route_stats' option is deprecated. " \
-        "Use 'performance_stats_flush_period' instead"
+        "Use 'performance_stats' instead"
       )
       @performance_stats = value
     end

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

@@ -11,20 +11,24 @@ module Airbrake
       def initialize(root_directory)
         @git_path = File.join(root_directory, '.git')
         @repository = nil
-        @git_version = Gem::Version.new(`git --version`.split[2])
+        @git_version = detect_git_version
         @weight = 116
       end
 
       # @macro call_filter
       def call(notice)
         return if notice[:context].key?(:repository)
+        attach_repository(notice)
+      end
 
+      def attach_repository(notice)
         if @repository
           notice[:context][:repository] = @repository
           return
         end
 
         return unless File.exist?(@git_path)
+        return unless @git_version
 
         @repository =
           if @git_version >= Gem::Version.new('2.7.0')
@@ -37,6 +41,24 @@ module Airbrake
         return unless @repository
         notice[:context][:repository] = @repository
       end
+
+      private
+
+      def detect_git_version
+        return unless which('git')
+        Gem::Version.new(`git --version`.split[2])
+      end
+
+      # Cross-platform way to tell if an executable is accessible.
+      def which(cmd)
+        exts = ENV['PATHEXT'] ? ENV['PATHEXT'].split(';') : ['']
+        ENV['PATH'].split(File::PATH_SEPARATOR).find do |path|
+          exts.find do |ext|
+            exe = File.join(path, "#{cmd}#{ext}")
+            File.executable?(exe) && !File.directory?(exe)
+          end
+        end
+      end
     end
   end
 end

data/lib/airbrake-ruby/version.rb

@@ -2,5 +2,5 @@
 # More information: http://semver.org/
 module Airbrake
   # @return [String] the library version
-  AIRBRAKE_RUBY_VERSION = '3.2.3'.freeze
+  AIRBRAKE_RUBY_VERSION = '3.2.4'.freeze
 end

data/spec/filters/git_repository_filter.rb

@@ -50,4 +50,12 @@ RSpec.describe Airbrake::Filters::GitRepositoryFilter do
       )
     end
   end
+
+  context "when git is not in PATH" do
+    it "does not attach context/repository" do
+      ENV['PATH'] = ''
+      subject.call(notice)
+      expect(notice[:context][:repository]).to be_nil
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: airbrake-ruby
 version: !ruby/object:Gem::Version
-  version: 3.2.3
+  version: 3.2.4
 platform: ruby
 authors:
 - Airbrake Technologies, Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-02-12 00:00:00.000000000 Z
+date: 2019-02-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rbtree3