honeybadger 1.16.2 → 1.16.3

data/CHANGELOG.md

@@ -1,3 +1,20 @@
+## Honeybadger 1.16.3 ##
+
+* Handle `nil` session in Rails controllers (i.e., when the `session_off` gem is
+  enabled).
+
+  *Joshua Wood*
+
+* Add `unwrap_exceptions` configuration option; when `false`, exceptions will
+  not be unwrapped (default: `true`).
+
+  *Joshua Wood*
+
+* Prevent `binding_of_caller` from being loaded automatically (requires
+  `send_local_variables == true`).
+
+  *Joshua Wood*
+
 ## Honeybadger 1.16.2 ##
 
 * Fix a bug where #match was called on nil when instrumenting Net::HTTP

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    honeybadger (1.16.2)
+    honeybadger (1.16.3)
       json
 
 GEM

data/features/step_definitions/rails_steps.rb

@@ -86,7 +86,7 @@ Then /^I should receive (.+) Honeybadger notifications?$/ do |number|
            when 'two' then 2
            else number end
 
-  all_output.scan(/\[Honeybadger\] Response from Honeybadger:/).size.should == number
+  all_output.scan(/\[Honeybadger\] Success: Net::HTTPOK/).size.should == number
   step %(the output should contain "123456789")
 end
 

data/honeybadger.gemspec

@@ -4,8 +4,8 @@ Gem::Specification.new do |s|
   s.rubygems_version = '1.3.5'
 
   s.name              = 'honeybadger'
-  s.version           = '1.16.2'
-  s.date              = '2014-07-16'
+  s.version           = '1.16.3'
+  s.date              = '2014-07-21'
 
   s.summary     = 'Error reports you can be happy about.'
   s.description = 'Make managing application errors a more pleasant experience.'
@@ -96,10 +96,10 @@ Gem::Specification.new do |s|
     lib/honeybadger/capistrano/tasks.rake
     lib/honeybadger/configuration.rb
     lib/honeybadger/dependency.rb
-    lib/honeybadger/exception_extensions.rb
     lib/honeybadger/integrations.rb
     lib/honeybadger/integrations/delayed_job.rb
     lib/honeybadger/integrations/delayed_job/plugin.rb
+    lib/honeybadger/integrations/local_variables.rb
     lib/honeybadger/integrations/net_http.rb
     lib/honeybadger/integrations/passenger.rb
     lib/honeybadger/integrations/sidekiq.rb
@@ -140,8 +140,8 @@ Gem::Specification.new do |s|
     spec/honeybadger/capistrano_spec.rb
     spec/honeybadger/configuration_spec.rb
     spec/honeybadger/dependency_spec.rb
-    spec/honeybadger/exception_extensions_spec.rb
     spec/honeybadger/integrations/delayed_job_spec.rb
+    spec/honeybadger/integrations/local_variables_spec.rb
     spec/honeybadger/integrations/net_http_spec.rb
     spec/honeybadger/integrations/passenger_spec.rb
     spec/honeybadger/integrations/sidekiq_spec.rb

data/lib/honeybadger.rb

@@ -14,13 +14,12 @@ require 'honeybadger/stats'
 require 'honeybadger/user_informer'
 require 'honeybadger/user_feedback'
 require 'honeybadger/integrations'
-require 'honeybadger/exception_extensions'
 
 require 'honeybadger/railtie' if defined?(Rails::Railtie)
 require 'honeybadger/monitor'
 
 module Honeybadger
-  VERSION = '1.16.2'.freeze
+  VERSION = '1.16.3'.freeze
   LOG_PREFIX = "** [Honeybadger] ".freeze
 
   HEADERS = {
@@ -191,6 +190,7 @@ module Honeybadger
     end
 
     def unwrap_exception(exception)
+      return exception unless configuration.unwrap_exceptions
       exception.respond_to?(:original_exception) && exception.original_exception ||
       exception.respond_to?(:continued_exception) && exception.continued_exception ||
       exception.respond_to?(:cause) && exception.cause ||

data/lib/honeybadger/configuration.rb

@@ -9,7 +9,8 @@ module Honeybadger
                :proxy_port, :proxy_user, :secure, :use_system_ssl_cert_chain, :framework,
                :user_information, :feedback, :rescue_rake_exceptions, :source_extract_radius,
                :send_request_session, :debug, :fingerprint, :hostname, :features, :metrics,
-               :log_exception_on_send_failure, :send_local_variables, :traces, :trace_threshold].freeze
+               :log_exception_on_send_failure, :send_local_variables, :traces,
+               :trace_threshold, :unwrap_exceptions].freeze
 
     # The API key for your project, found on the project edit form.
     attr_accessor :api_key
@@ -107,6 +108,9 @@ module Honeybadger
     # +true+ to send local variables, +false+ to exclude
     attr_accessor :send_local_variables
 
+    # +true+ to unwrap exceptions
+    attr_accessor :unwrap_exceptions
+
     # +true+ to log extra debug info, +false+ to suppress
     attr_accessor :debug
 
@@ -195,6 +199,7 @@ module Honeybadger
       @limit                         = nil
       @feedback                      = true
       @trace_threshold               = 2000
+      @unwrap_exceptions             = true
     end
 
     # Public: Takes a block and adds it to the list of backtrace filters. When

data/lib/honeybadger/integrations.rb

@@ -6,3 +6,4 @@ require 'honeybadger/integrations/thor'
 require 'honeybadger/integrations/net_http'
 require 'honeybadger/integrations/passenger'
 require 'honeybadger/integrations/unicorn'
+require 'honeybadger/integrations/local_variables'

data/lib/honeybadger/integrations/local_variables.rb

@@ -0,0 +1,36 @@
+module Honeybadger
+  module Integrations
+    module LocalVariables
+      module ExceptionExtension
+        def self.included(base)
+          base.send(:alias_method, :set_backtrace_without_honeybadger, :set_backtrace)
+          base.send(:alias_method, :set_backtrace, :set_backtrace_with_honeybadger)
+        end
+
+        def set_backtrace_with_honeybadger(*args, &block)
+          if caller.none? { |loc| loc.match(::Honeybadger::Backtrace::Line::INPUT_FORMAT) && Regexp.last_match(1) == __FILE__ }
+            @__honeybadger_bindings_stack = binding.callers.drop(1)
+          end
+
+          set_backtrace_without_honeybadger(*args, &block)
+        end
+
+        def __honeybadger_bindings_stack
+          @__honeybadger_bindings_stack || []
+        end
+      end
+
+      Dependency.register do
+        requirement { ::Honeybadger.configuration.send_local_variables }
+        requirement { defined?(::BindingOfCaller) }
+        requirement { !::Exception.included_modules.include?(ExceptionExtension) }
+
+        injection { Honeybadger.write_verbose_log('Installing local variables integration') }
+
+        injection do
+          ::Exception.send(:include, ExceptionExtension)
+        end
+      end
+    end
+  end
+end

data/lib/honeybadger/monitor/sender.rb

@@ -8,7 +8,7 @@ module Honeybadger
         response = send_request('/v1/metrics', data.to_json)
 
         if Net::HTTPSuccess === response
-          log(:info, "Metrics Success: #{response.class}", response, data) if Honeybadger.configuration.debug
+          log(:info, "Metrics Success: #{response.class}", response) if Honeybadger.configuration.debug
           true
         else
           Honeybadger.configuration.features['metrics'] = false if Net::HTTPForbidden === response
@@ -27,7 +27,7 @@ module Honeybadger
         response = send_request('/v1/traces', data.to_json)
 
         if Net::HTTPSuccess === response
-          log(:info, "Traces Success: #{response.class}", response, data) if Honeybadger.configuration.debug
+          log(:info, "Traces Success: #{response.class}", response) if Honeybadger.configuration.debug
           true
         else
           Honeybadger.configuration.features['traces'] = false if Net::HTTPForbidden === response

data/lib/honeybadger/notice.rb

@@ -402,6 +402,7 @@ module Honeybadger
     # Returns a Hash of local variables
     def local_variables_from_exception(exception)
       return {} unless Exception === exception
+      return {} unless exception.respond_to?(:__honeybadger_bindings_stack)
       return {} if exception.__honeybadger_bindings_stack.empty?
 
       if project_root

data/lib/honeybadger/rails/controller_methods.rb

@@ -55,6 +55,7 @@ module Honeybadger
       end
 
       def honeybadger_session_data
+        return {} unless session
         if session.respond_to?(:to_hash)
           session.to_hash
         else

data/lib/honeybadger/sender.rb

@@ -68,10 +68,11 @@ module Honeybadger
     def ping(data = {})
       return nil unless api_key_ok?
 
-      response = send_request('/v1/ping/', data.to_json)
+      data = data.to_json
+      response = send_request('/v1/ping/', data)
 
       if Net::HTTPSuccess === response
-        log(Honeybadger.configuration.debug ? :info : :debug, "Ping Success: #{response.class}", response, data)
+        log(Honeybadger.configuration.debug ? :info : :debug, "Ping Success: #{response.class}", response)
         JSON.parse(response.body)
       else
         log(:error, "Ping Failure: #{response.class}", response, data)

data/spec/honeybadger/configuration_spec.rb

@@ -31,6 +31,7 @@ describe Honeybadger::Configuration do
     assert_config_default :async, nil
     assert_config_default :send_request_session, true
     assert_config_default :send_local_variables, false
+    assert_config_default :unwrap_exceptions, true
     assert_config_default :debug, false
     assert_config_default :log_exception_on_send_failure, false
     assert_config_default :fingerprint, nil
@@ -138,7 +139,7 @@ describe Honeybadger::Configuration do
      :proxy_host, :proxy_pass, :proxy_port, :proxy_user, :secure,
      :source_extract_radius, :async, :send_request_session, :debug,
      :fingerprint, :hostname, :features, :metrics, :feedback,
-     :log_exception_on_send_failure].each do |option|
+     :log_exception_on_send_failure, :unwrap_exceptions].each do |option|
        expect(hash[option]).to eq config[option]
     end
   end

data/spec/honeybadger/integrations/local_variables_spec.rb

@@ -0,0 +1,60 @@
+require 'spec_helper'
+
+describe "Local variables integration" do
+  before do
+    Honeybadger.configuration.send_local_variables = config_enabled
+    Honeybadger::Dependency.inject!
+  end
+
+  subject { Exception.new }
+
+  context "when binding_of_caller isn't installed", :unless => defined?(::BindingOfCaller) do
+    let(:config_enabled) { true }
+    it { should_not respond_to :__honeybadger_bindings_stack }
+  end
+
+  context "when binding_of_caller is installed", :if => defined?(::BindingOfCaller) do
+    context "and disabled by configuration" do
+      let(:config_enabled) { false }
+      it { should_not respond_to :__honeybadger_bindings_stack }
+    end
+
+    context "and enabled by configuration" do
+      let(:config_enabled) { true }
+
+      it { should respond_to :__honeybadger_bindings_stack }
+
+      describe "#set_backtrace" do
+        context "call stack does not match current file" do
+          it "changes the bindings stack" do
+            expect { subject.set_backtrace(['foo.rb:1']) }.to change(subject, :__honeybadger_bindings_stack).from([])
+          end
+        end
+
+        context "call stack includes current file" do
+          before do
+            subject.stub(:caller).and_return(["#{File.expand_path('../../../../lib/honeybadger/integrations/local_variables.rb', __FILE__)}:1"])
+          end
+
+          it "does not change the bindings stack" do
+            expect { subject.set_backtrace(['foo.rb:1']) }.not_to change(subject, :__honeybadger_bindings_stack).from([])
+          end
+        end
+
+        context "call stack includes a non-matching line" do
+          before do
+            subject.stub(:caller).and_return(['(foo)'])
+          end
+
+          it "skips the non-matching line" do
+            expect { subject.set_backtrace(['foo.rb:1']) }.not_to raise_error
+          end
+
+          it "changes the bindings stack" do
+            expect { subject.set_backtrace(['foo.rb:1']) }.to change(subject, :__honeybadger_bindings_stack).from([])
+          end
+        end
+      end
+    end
+  end
+end

data/spec/honeybadger/notifier_spec.rb

@@ -233,7 +233,7 @@ describe 'Honeybadger' do
     config_opts = { 'one' => 'two', 'three' => 'four' }
     stub_notice!
     stub_sender!
-    Honeybadger.configuration = double('config', :merge => config_opts, :public? => true, :async? => false)
+    Honeybadger.configuration = double('config', :merge => config_opts, :public? => true, :async? => false, :unwrap_exceptions => false)
 
     Honeybadger::Notice.should_receive(:new).with(hash_including(config_opts))
     Honeybadger.notify(exception)
@@ -299,6 +299,15 @@ describe 'Honeybadger' do
         @hash = Honeybadger.build_lookup_hash_for(@exception)
         expect(@hash[:error_class]).to eq "OriginalException"
       end
+
+      context "but unwrapping exceptions is disabled" do
+        before { Honeybadger.configuration.unwrap_exceptions = false }
+
+        it "doesn't unwrap the exception" do
+          @hash = Honeybadger.build_lookup_hash_for(@exception)
+          expect(@hash[:error_class]).not_to eq "OriginalException"
+        end
+      end
     end
 
     context "when an exception that provides #continued_exception is raised" do

data/spec/honeybadger/rails/action_controller_spec.rb

@@ -131,7 +131,11 @@ begin
       end
 
       opts[:request].query_parameters = opts[:request].query_parameters.merge(opts[:params] || {})
-      opts[:request].session = ActionController::TestSession.new(opts[:session] || {})
+      opts[:request].session = if opts.include?(:session) && opts[:session].nil?
+                                 nil
+                               else
+                                 ActionController::TestSession.new(opts[:session] || {})
+                               end
       # Prevents request.fullpath from crashing Rails in tests
       opts[:request].env['REQUEST_URI'] = opts[:request].request_uri
 
@@ -240,6 +244,11 @@ begin
       assert_sent_hash(data) { |h| h['request']['session'] }
     end
 
+    it "fails gracefully when session isn nil" do
+      process_action_with_automatic_notification(:session => nil)
+      assert_sent_hash({}) { |h| h['request']['session'] }
+    end
+
     it "send request data for manual notification" do
       params = { 'controller' => "honeybadger_test", 'action' => "index" }
       controller = process_action_with_manual_notification(:params => params)

data/spec/spec_helper.rb

@@ -1,6 +1,12 @@
 require 'rspec'
 require 'honeybadger'
 
+begin
+  require 'binding_of_caller'
+rescue LoadError
+  nil
+end
+
 # Require files in spec/support/ and its subdirectories.
 Dir[File.expand_path('../../spec/support/**/*.rb', __FILE__)].each {|f| require f}
 

metadata

@@ -1,195 +1,222 @@
 --- !ruby/object:Gem::Specification
 name: honeybadger
 version: !ruby/object:Gem::Version
-  version: 1.16.2
+  version: 1.16.3
+  prerelease: 
 platform: ruby
 authors:
 - Joshua Wood
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-07-16 00:00:00.000000000 Z
+date: 2014-07-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: cucumber
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: 1.3.10
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: 1.3.10
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: 2.14.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: 2.14.0
 - !ruby/object:Gem::Dependency
   name: sham_rack
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: 1.3.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: 1.3.0
 - !ruby/object:Gem::Dependency
   name: capistrano
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '2.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '2.0'
 - !ruby/object:Gem::Dependency
   name: guard
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: 1.8.3
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: 1.8.3
 - !ruby/object:Gem::Dependency
   name: guard-rspec
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: sinatra
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: aruba
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: appraisal
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: fuubar
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: growl
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 description: Make managing application errors a more pleasant experience.
@@ -257,10 +284,10 @@ files:
 - lib/honeybadger/capistrano/tasks.rake
 - lib/honeybadger/configuration.rb
 - lib/honeybadger/dependency.rb
-- lib/honeybadger/exception_extensions.rb
 - lib/honeybadger/integrations.rb
 - lib/honeybadger/integrations/delayed_job.rb
 - lib/honeybadger/integrations/delayed_job/plugin.rb
+- lib/honeybadger/integrations/local_variables.rb
 - lib/honeybadger/integrations/net_http.rb
 - lib/honeybadger/integrations/passenger.rb
 - lib/honeybadger/integrations/sidekiq.rb
@@ -301,8 +328,8 @@ files:
 - spec/honeybadger/capistrano_spec.rb
 - spec/honeybadger/configuration_spec.rb
 - spec/honeybadger/dependency_spec.rb
-- spec/honeybadger/exception_extensions_spec.rb
 - spec/honeybadger/integrations/delayed_job_spec.rb
+- spec/honeybadger/integrations/local_variables_spec.rb
 - spec/honeybadger/integrations/net_http_spec.rb
 - spec/honeybadger/integrations/passenger_spec.rb
 - spec/honeybadger/integrations/sidekiq_spec.rb
@@ -330,26 +357,30 @@ files:
 - spec/support/helpers.rb
 homepage: http://www.honeybadger.io
 licenses: []
-metadata: {}
 post_install_message: 
 rdoc_options:
-- "--charset=UTF-8"
-- "--markup tomdoc"
+- --charset=UTF-8
+- --markup tomdoc
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
+  none: false
   requirements:
-  - - ">="
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
+      segments:
+      - 0
+      hash: 3175357828774041606
 required_rubygems_version: !ruby/object:Gem::Requirement
+  none: false
   requirements:
-  - - ">="
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 1.8.23
 signing_key: 
 specification_version: 2
 summary: Error reports you can be happy about.

checksums.yaml

@@ -1,7 +0,0 @@
----
-SHA1:
-  metadata.gz: 86f0d9ed43c1f392c2eed8701f7dcc311df00c2e
-  data.tar.gz: 2154070bb26a86dd27e17d7893748ebbd7838d37
-SHA512:
-  metadata.gz: eb0c4089da555f08d4f753abb25809fa1c6def54a276768d1b6e4a14c326da46a2069108caa8a115efb328250670137929e9229b8d190f24db69845f740d151b
-  data.tar.gz: 9d298e5339abc7a3ebd7a7c53e8d35b606b007d30c092817b6d3a4c38145a8d26df74701344c21a0821ab3e40617dd5d2244e7b25fbd3dce75423ee687384f92

data/lib/honeybadger/exception_extensions.rb

@@ -1,35 +0,0 @@
-module Honeybadger
-  module ExceptionExtensions
-    module Bindings
-      def self.included(base)
-        base.send(:alias_method, :set_backtrace_without_honeybadger, :set_backtrace)
-        base.send(:alias_method, :set_backtrace, :set_backtrace_with_honeybadger)
-      end
-
-      def set_backtrace_with_honeybadger(*args, &block)
-        if caller.none? { |loc| loc.match(Honeybadger::Backtrace::Line::INPUT_FORMAT) && Regexp.last_match(1) == __FILE__ }
-          @__honeybadger_bindings_stack = binding.callers.drop(1)
-        end
-
-        set_backtrace_without_honeybadger(*args, &block)
-      end
-
-      def __honeybadger_bindings_stack
-        @__honeybadger_bindings_stack || []
-      end
-    end
-
-    module NullBindings
-      def __honeybadger_bindings_stack
-        []
-      end
-    end
-  end
-end
-
-begin
-  require 'binding_of_caller'
-  Exception.send(:include, Honeybadger::ExceptionExtensions::Bindings)
-rescue LoadError
-  Exception.send(:include, Honeybadger::ExceptionExtensions::NullBindings)
-end

data/spec/honeybadger/exception_extensions_spec.rb

@@ -1,40 +0,0 @@
-require 'spec_helper'
-
-describe Exception, :unless => defined?(BindingOfCaller) do
-  should { respond_to :__honeybadger_bindings_stack }
-  its(:__honeybadger_bindings_stack) { should eq([]) }
-end
-
-describe Exception, :if => defined?(BindingOfCaller) do
-  describe "#set_backtrace" do
-    context "call stack does not match current file" do
-      it "changes the bindings stack" do
-        expect { subject.set_backtrace(['foo.rb:1']) }.to change(subject, :__honeybadger_bindings_stack).from([])
-      end
-    end
-
-    context "call stack includes current file" do
-      before do
-        subject.stub(:caller).and_return(["#{File.expand_path('../../../lib/honeybadger/exception_extensions.rb', __FILE__)}:1"])
-      end
-
-      it "does not change the bindings stack" do
-        expect { subject.set_backtrace(['foo.rb:1']) }.not_to change(subject, :__honeybadger_bindings_stack).from([])
-      end
-    end
-
-    context "call stack includes a non-matching line" do
-      before do
-        subject.stub(:caller).and_return(['(foo)'])
-      end
-
-      it "skips the non-matching line" do
-        expect { subject.set_backtrace(['foo.rb:1']) }.not_to raise_error
-      end
-
-      it "changes the bindings stack" do
-        expect { subject.set_backtrace(['foo.rb:1']) }.to change(subject, :__honeybadger_bindings_stack).from([])
-      end
-    end
-  end
-end