honeybadger 4.5.4 → 4.7.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c2e3e46ae65528005cf9ff22cb073c92a4c159b316bacc040907bdff79828a42
-  data.tar.gz: 788b73aab89e83e52b1d94c8d18f3ce3a9232698412ac11bd16be91ce7011cc1
+  metadata.gz: 2c76ef937ae77d449421f97e342bd86f77176a705fb6b7a75e4928e0a48fc71b
+  data.tar.gz: db036be307a2f741187951a46f30845451e5e01d49d266250837dcb6869b084c
 SHA512:
-  metadata.gz: d4304b0457b18a8b7ed6748e453e99979cc496dff52bbae872940d3341f6aaa642214966dbffd9c73a0d0170ebdaf84ce00af891a1e24deaeaa1bc064d6ad557
-  data.tar.gz: e1b3d6a70a1ae058380c4ff4fbb65722475c431879cb5add603e2a0b5efef7a6ed4de85204af03ecf020afb74b94d682d221a02a2c6ab5b2337289e263becc61
+  metadata.gz: e57d7371fa97c9e9f356824ac5ee15454106cbf332d772c0fc74f01a5d8d5aaac7b78f7f09eacc2d4e902c432f24ca30146ed36e697048ee9a9e31902472942d
+  data.tar.gz: 6f14009e5ea56d72bdba4a95a3bcfe23fd29e6bc8870b701e3f6a0869e4aa750bc56485ac149f512484881bdd477af7c8dd41668b104323d54be840c31f13800

data/CHANGELOG.md

@@ -5,6 +5,43 @@ adheres to [Semantic Versioning](http://semver.org/).
 
 ## [Unreleased]
 
+## [4.7.1] - 2020-08-11
+### Fixed
+- ActiveRecord SQL Breadcrumb event pulls adapter from supplied connection,
+  allowing for multiple databases.
+- Fix Rails deprecation of `ActionDispatch::ParamsParser::ParseError`
+- Deal with invalid UTF-8 byte sequences during SQL obfuscation
+- Fix Ruby 2.7 deprecation notice in sql.rb
+
+## [4.7.0] - 2020-06-02
+### Fixed
+- Alias `Notice#controller=` as `Notice#component=`
+- Fix Rails 6.1 deprecation warning with `ActiveRecord::Base.connection_config`
+- Fix agent where breadcrumbs.enabled = true and local_context = true
+
+### Added
+- Add `honeybadger_skip_rails_load` Capistrano option to skip rails load on
+  deployment notification (#355) -@NielsKSchjoedt
+
+## [4.6.0] - 2020-03-12
+### Fixed
+- Fixed issue where Sidekiq.attempt_threshold was triggering 2 attempts ahead
+  of the setting
+- Dup notify opts before mutating (#345)
+
+### Changed
+- Breadcrumbs on by default
+- Added Faktory plugin -@scottrobertson
+
+## [4.5.6] - 2020-01-08
+### Fixed
+- Fix remaining Ruby 2.7 deprecation warnings
+
+## [4.5.5] - 2020-01-06
+### Fixed
+- Replace empty `Proc.new` with explicit block param to suppress warnings
+  in Ruby 2.7
+
 ## [4.5.4] - 2019-12-09
 ### Fixed
 - Re-released to remove vendor cruft

data/README.md

@@ -1,6 +1,7 @@
 # Honeybadger for Ruby
 
-[![CircleCI](https://circleci.com/gh/honeybadger-io/honeybadger-ruby.svg?style=svg)](https://circleci.com/gh/honeybadger-io/honeybadger-ruby)
+![Ruby](https://github.com/honeybadger-io/honeybadger-ruby/workflows/Ruby/badge.svg)
+![JRuby](https://github.com/honeybadger-io/honeybadger-ruby/workflows/JRuby/badge.svg)
 [![Gem Version](https://badge.fury.io/rb/honeybadger.svg)](http://badge.fury.io/rb/honeybadger)
 [![SemVer](https://api.dependabot.com/badges/compatibility_score?dependency-name=honeybadger&package-manager=bundler&version-scheme=semver)](https://dependabot.com/compatibility-score.html?dependency-name=honeybadger&package-manager=bundler&version-scheme=semver)
 

data/lib/honeybadger/agent.rb

@@ -63,13 +63,17 @@ module Honeybadger
       end
 
       @context = opts.delete(:context)
-      if opts.delete(:local_context)
+      local_context = opts.delete(:local_context)
+
+      @config ||= Config.new(opts)
+
+      if local_context
         @context ||= ContextManager.new
         @breadcrumbs = Breadcrumbs::Collector.new(config)
+      else
+        @breadcrumbs = nil
       end
 
-      @config ||= Config.new(opts)
-
       init_worker
     end
 
@@ -115,6 +119,8 @@ module Honeybadger
     # @return [String] UUID reference to the notice within Honeybadger.
     # @return [false] when ignored.
     def notify(exception_or_opts, opts = {})
+      opts = opts.dup
+
       if exception_or_opts.is_a?(Exception)
         opts[:exception] = exception_or_opts
       elsif exception_or_opts.respond_to?(:to_hash)
@@ -278,7 +284,7 @@ module Honeybadger
         metadata: metadata
       })
 
-      breadcrumbs.add!(Breadcrumbs::Breadcrumb.new(params))
+      breadcrumbs.add!(Breadcrumbs::Breadcrumb.new(**params))
 
       self
     end

data/lib/honeybadger/breadcrumbs/active_support.rb

@@ -15,10 +15,11 @@ module Honeybadger
               ["Active Record", name].compact.join(" - ")
             end,
             category: "query",
-            select_keys: [:sql, :name, :connection_id, :cached],
+            select_keys: [:sql, :name, :connection, :connection_id, :cached],
             transform: lambda do |data|
               if data[:sql]
-                adapter = ::ActiveRecord::Base.connection_config[:adapter]
+                connection = data.delete(:connection) || ::ActiveRecord::Base.connection
+                adapter = connection.adapter_name.downcase
                 data[:sql] = Util::SQL.obfuscate(data[:sql], adapter)
               end
               data
@@ -102,7 +103,6 @@ module Honeybadger
           }
         }
       end
-
     end
   end
 end

data/lib/honeybadger/cli/helpers.rb

@@ -15,7 +15,7 @@ To fix this issue, please try the following:
 
   - Make sure the gem is configured properly.
   - Retry executing this command a few times.
-  - Make sure you can connect to #{host} (`ping #{host}`).
+  - Make sure you can connect to #{host} (`curl https://#{host}/v1/notices`).
   - Email support@honeybadger.io for help. Include as much debug info as you
     can for a faster resolution!
 

data/lib/honeybadger/cli/main.rb

@@ -196,7 +196,7 @@ WELCOME
 # What can I do?
 
   - Retry the command.
-  - Make sure you can connect to api.honeybadger.io (`ping api.honeybadger.io`).
+  - Make sure you can connect to api.honeybadger.io (`curl https://api.honeybadger.io/v1/notices`).
   - If you continue to see this message, email us at support@honeybadger.io
     (don't forget to attach this output!)
 

data/lib/honeybadger/cli/test.rb

@@ -175,7 +175,7 @@ To fix this issue, please try the following:
 
   - Make sure the gem is configured properly.
   - Retry executing this command a few times.
-  - Make sure you can connect to #{host} (`ping #{host}`).
+  - Make sure you can connect to #{host} (`curl https://#{host}/v1/notices`).
   - Email support@honeybadger.io for help. Include as much debug info as you
     can for a faster resolution!
 

data/lib/honeybadger/config.rb

@@ -79,10 +79,10 @@ module Honeybadger
       self
     end
 
-    def backtrace_filter
+    def backtrace_filter(&block)
       if block_given?
         warn('DEPRECATED: backtrace_filter is deprecated. Please use before_notify instead. See https://docs.honeybadger.io/ruby/support/v4-upgrade#backtrace_filter')
-        self[:backtrace_filter] = Proc.new
+        self[:backtrace_filter] = block
       end
 
       self[:backtrace_filter]
@@ -92,19 +92,19 @@ module Honeybadger
       (ruby[:before_notify] || []).clone
     end
 
-    def exception_filter
+    def exception_filter(&block)
       if block_given?
         warn('DEPRECATED: exception_filter is deprecated. Please use before_notify instead. See https://docs.honeybadger.io/ruby/support/v4-upgrade#exception_filter')
-        self[:exception_filter] = Proc.new
+        self[:exception_filter] = block
       end
 
       self[:exception_filter]
     end
 
-    def exception_fingerprint
+    def exception_fingerprint(&block)
       if block_given?
         warn('DEPRECATED: exception_fingerprint is deprecated. Please use before_notify instead. See https://docs.honeybadger.io/ruby/support/v4-upgrade#exception_fingerprint')
-        self[:exception_fingerprint] = Proc.new
+        self[:exception_fingerprint] = block
       end
 
       self[:exception_fingerprint]

data/lib/honeybadger/config/defaults.rb

@@ -13,7 +13,12 @@ module Honeybadger
                       'ActionController::UnknownFormat',
                       'ActionController::InvalidAuthenticityToken',
                       'ActionController::InvalidCrossOriginRequest',
+                      # ActionDispatch::ParamsParser::ParseError was removed in Rails 6.0
+                      # and may be removed here once support for Rails 5.2 is dropped.
+                      # https://github.com/rails/rails/commit/e16c765ac6dcff068ff2e5554d69ff345c003de1
+                      # https://github.com/honeybadger-io/honeybadger-ruby/pull/358
                       'ActionDispatch::ParamsParser::ParseError',
+                      'ActionDispatch::Http::Parameters::ParseError',
                       'ActionController::BadRequest',
                       'ActionController::ParameterMissing',
                       'ActiveRecord::RecordNotFound',
@@ -279,6 +284,11 @@ module Honeybadger
         default: 0,
         type: Integer
       },
+      :'faktory.attempt_threshold' => {
+        description: 'The number of attempts before notifications will be sent.',
+        default: 0,
+        type: Integer
+      },
       :'sidekiq.use_component' => {
         description: 'Automatically set the component to the class of the job. Helps with grouping.',
         default: true,
@@ -295,8 +305,8 @@ module Honeybadger
         type: Boolean
       },
       :'breadcrumbs.enabled' => {
-        description: 'Enable/Disable breadcrumb functionality.',
-        default: false,
+        description: 'Disable breadcrumb functionality.',
+        default: true,
         type: Boolean
       },
       :'breadcrumbs.active_support_notifications' => {

data/lib/honeybadger/config/ruby.rb

@@ -93,28 +93,28 @@ module Honeybadger
         hash[:before_notify] = hooks
       end
 
-      def backtrace_filter
+      def backtrace_filter(&block)
         if block_given?
           logger.warn('DEPRECATED: backtrace_filter is deprecated. Please use before_notify instead. See https://docs.honeybadger.io/ruby/support/v4-upgrade#backtrace_filter')
-          hash[:backtrace_filter] = Proc.new if block_given?
+          hash[:backtrace_filter] = block if block_given?
         end
 
         get(:backtrace_filter)
       end
 
-      def exception_filter
+      def exception_filter(&block)
         if block_given?
           logger.warn('DEPRECATED: exception_filter is deprecated. Please use before_notify instead. See https://docs.honeybadger.io/ruby/support/v4-upgrade#exception_filter')
-          hash[:exception_filter] = Proc.new
+          hash[:exception_filter] = block
         end
 
         get(:exception_filter)
       end
 
-      def exception_fingerprint
+      def exception_fingerprint(&block)
         if block_given?
           logger.warn('DEPRECATED: exception_fingerprint is deprecated. Please use before_notify instead. See https://docs.honeybadger.io/ruby/support/v4-upgrade#exception_fingerprint')
-          hash[:exception_fingerprint] = Proc.new
+          hash[:exception_fingerprint] = block
         end
 
         get(:exception_fingerprint)

data/lib/honeybadger/notice.rb

@@ -42,6 +42,10 @@ module Honeybadger
   # @api private
   MAX_EXCEPTION_CAUSES = 5
 
+  # @api private
+  # Binding#source_location was added in Ruby 2.6.
+  BINDING_HAS_SOURCE_LOCATION = Binding.method_defined?(:source_location)
+
   class Notice
     extend Forwardable
 
@@ -114,6 +118,7 @@ module Honeybadger
     # The component (if any) which was used in this request (usually the controller).
     attr_accessor :component
     alias_method :controller, :component
+    alias_method :controller=, :component=
 
     # The action (if any) that was called in this request.
     attr_accessor :action
@@ -444,7 +449,13 @@ module Honeybadger
       return {} if exception.__honeybadger_bindings_stack.empty?
 
       if config[:root]
-        binding = exception.__honeybadger_bindings_stack.find { |b| b.eval('__FILE__') =~ /^#{Regexp.escape(config[:root].to_s)}/ }
+        binding = exception.__honeybadger_bindings_stack.find { |b|
+          if BINDING_HAS_SOURCE_LOCATION
+            b.source_location[0]
+          else
+            b.eval('__FILE__')
+          end =~ /^#{Regexp.escape(config[:root].to_s)}/
+        }
       end
 
       binding ||= exception.__honeybadger_bindings_stack[0]

data/lib/honeybadger/plugin.rb

@@ -66,11 +66,11 @@ module Honeybadger
       #   +snake_case+. The name is inferred from the current file name if omitted.
       #
       # @return nil
-      def register(name = nil)
+      def register(name = nil, &block)
         name ||= name_from_caller(caller) or
           raise(ArgumentError, 'Plugin name is required, but was nil.')
         instances[key = name.to_sym] and fail("Already registered: #{name}")
-        instances[key] = new(name).tap { |d| d.instance_eval(&Proc.new) }
+        instances[key] = new(name).tap { |d| d.instance_eval(&block) }
       end
 
       # @api private
@@ -136,8 +136,8 @@ module Honeybadger
     #   end
     #
     # @return nil
-    def requirement
-      @requirements << Proc.new
+    def requirement(&block)
+      @requirements << block
     end
 
     # Define an execution block. Execution blocks will be executed if all
@@ -161,8 +161,8 @@ module Honeybadger
     #   end
     #
     # @return nil
-    def execution
-      @executions << Proc.new
+    def execution(&block)
+      @executions << block
     end
 
     # @api private

data/lib/honeybadger/plugins/faktory.rb

@@ -0,0 +1,52 @@
+require 'honeybadger/plugin'
+require 'honeybadger/ruby'
+
+module Honeybadger
+  module Plugins
+    module Faktory
+      class Middleware
+        def call(worker, job)
+          Honeybadger.clear!
+          yield
+        end
+      end
+
+      Plugin.register do
+        requirement { defined?(::Faktory) }
+
+        execution do
+          ::Faktory.configure_worker do |faktory|
+            faktory.worker_middleware do |chain|
+              chain.prepend Middleware
+            end
+          end
+
+          ::Faktory.configure_worker do |faktory|
+            faktory.error_handlers << lambda do |ex, params|
+              opts = {parameters: params}
+
+              if job = params[:job]
+                if (threshold = config[:'faktory.attempt_threshold'].to_i) > 0
+                  # If job.failure is nil, it is the first attempt. The first
+                  # retry has a job.failure.retry_count of 0, which would be
+                  # the second attempt in our case.
+                  retry_count = job.dig('failure', 'retry_count')
+                  attempt = retry_count ? retry_count + 1 : 0
+
+                  limit = [job['retry'].to_i, threshold].min
+
+                  return if attempt < limit
+                end
+
+                opts[:component] = job['jobtype']
+                opts[:action] = 'perform'
+              end
+
+              Honeybadger.notify(ex, opts)
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/honeybadger/plugins/sidekiq.rb

@@ -25,20 +25,30 @@ module Honeybadger
             ::Sidekiq.configure_server do |sidekiq|
               sidekiq.error_handlers << lambda {|ex, params|
                 job = params[:job] || params
-                retry_count = job['retry_count'.freeze].to_i
-                retry_opt = job['retry'.freeze]
-                max_retries = if retry_opt.is_a?(Integer)
-                  [retry_opt - 1, config[:'sidekiq.attempt_threshold'].to_i].min
-                else
-                  config[:'sidekiq.attempt_threshold'].to_i
+                job_retry = job['retry'.freeze]
+
+                if (threshold = config[:'sidekiq.attempt_threshold'].to_i) > 0 && job_retry
+                  # We calculate the job attempts to determine the need to
+                  # skip. Sidekiq's first job execution will have nil for the
+                  # 'retry_count' job key. The first retry will have 0 set for
+                  # the 'retry_count' key, incrementing on each execution
+                  # afterwards.
+                  retry_count = job['retry_count'.freeze]
+                  attempt = retry_count ? retry_count + 1 : 0
+
+                  # Ensure we account for modified max_retries setting
+                  retry_limit = job_retry == true ? (sidekiq.options[:max_retries] || 25) : job_retry.to_i
+                  limit = [retry_limit, threshold].min
+
+                  return if attempt < limit
                 end
 
-                return if retry_opt && retry_count < max_retries
                 opts = {parameters: params}
                 if config[:'sidekiq.use_component']
                   opts[:component] = job['wrapped'.freeze] || job['class'.freeze]
                   opts[:action] = 'perform' if opts[:component]
                 end
+
                 Honeybadger.notify(ex, opts)
               }
             end

data/lib/honeybadger/util/sanitizer.rb

@@ -187,7 +187,7 @@ module Honeybadger
 
       def valid_encoding(string)
         return string if valid_encoding?(string)
-        string.encode(Encoding::UTF_8, ENCODE_OPTS)
+        string.encode(Encoding::UTF_8, **ENCODE_OPTS)
       end
 
       def recursive?(data)

data/lib/honeybadger/util/sql.rb

@@ -11,7 +11,7 @@ module Honeybadger
       DoubleQuoters = /(postgres|sqlite|postgis)/.freeze
 
       def self.obfuscate(sql, adapter)
-        sql.dup.tap do |s|
+        force_utf_8(sql.dup).tap do |s|
           s.gsub!(EscapedQuotes, EmptyReplacement)
           s.gsub!(SQuotedData, Replacement)
           s.gsub!(DQuotedData, Replacement) if adapter =~ DoubleQuoters
@@ -20,6 +20,15 @@ module Honeybadger
           s.squeeze!(' ')
         end
       end
+
+      def self.force_utf_8(string)
+        string.encode(
+          Encoding.find('UTF-8'),
+          invalid: :replace, 
+          undef: :replace, 
+          replace: ''
+        )
+      end
     end
   end
 end

data/lib/honeybadger/version.rb

@@ -1,4 +1,4 @@
 module Honeybadger
   # The current String Honeybadger version.
-  VERSION = '4.5.4'.freeze
+  VERSION = '4.7.1'.freeze
 end

data/vendor/capistrano-honeybadger/lib/capistrano/tasks/deploy.cap

@@ -24,6 +24,7 @@ namespace :honeybadger do
 
         api_key = fetch(:honeybadger_api_key, ENV['HONEYBADGER_API_KEY'])
         options += ['--api-key', api_key] if api_key
+        options << '--skip-rails-load' if fetch(:honeybadger_skip_rails_load, false)
 
         if fetch(:honeybadger_async_notify, false)
           ::SSHKit.config.command_map.prefix[:honeybadger].push(:nohup)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: honeybadger
 version: !ruby/object:Gem::Version
-  version: 4.5.4
+  version: 4.7.1
 platform: ruby
 authors:
 - Honeybadger Industries LLC
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-12-10 00:00:00.000000000 Z
+date: 2020-08-11 00:00:00.000000000 Z
 dependencies: []
 description: Make managing application errors a more pleasant experience.
 email:
@@ -65,6 +65,7 @@ files:
 - lib/honeybadger/plugins/breadcrumbs.rb
 - lib/honeybadger/plugins/delayed_job.rb
 - lib/honeybadger/plugins/delayed_job/plugin.rb
+- lib/honeybadger/plugins/faktory.rb
 - lib/honeybadger/plugins/lambda.rb
 - lib/honeybadger/plugins/local_variables.rb
 - lib/honeybadger/plugins/passenger.rb
@@ -134,7 +135,7 @@ homepage: https://github.com/honeybadger-io/honeybadger-ruby
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options:
 - "--markup=tomdoc"
 - "--main=README.md"
@@ -152,8 +153,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.1.2
+signing_key:
 specification_version: 4
 summary: Error reports you can be happy about.
 test_files: []