raygun4ruby 2.6.0 → 2.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 12ad609c25703e38bf187602b1eeb27ed13f9fdb
-  data.tar.gz: 5af743e3db72f0339669b255328d8256be2c6314
+  metadata.gz: 1975c19107e042426b56c91ca986207e3e17d681
+  data.tar.gz: 491657544cbb5537c01fc4483dd45a8282add25c
 SHA512:
-  metadata.gz: 070d775307e84cd705e14c6c6a245d4508c8dfcb95991da06ef62e825255a12fde096808e27d65a11bc2c2e20e892270f3e005eb65a4c9280575075ed3a45bdc
-  data.tar.gz: 9b011bb6ca157c9fbcd0283721d40a48e90e857425c7055ad93f1445c98eb9c5103c3d7a6bbb2a6e18059bfbf2181d7b66e14c8a0a5ef65782ac84cf6dfcfbe9
+  metadata.gz: 79eb83863ba93f963857e2a43af01e9daff5e13cd788a218db867a5a622e51a8ab119b16f546534f7f12b680c9e086fa0d524502306d04b17c95461a56046ab0
+  data.tar.gz: 2a4fc829d54fe4deb855283719f7432360854760cf1d78f06235f82304587d00ed2607e42ff7b6cafe9def75e3b0585d81aa0930a403de5861e5e35f881fe967

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+## 2.7.0 (19/02/2018)
+
+Features
+ - Add configuration option to control network timeouts when sending error reports, default value is 10 seconds ([#129](https://github.com/MindscapeHQ/raygun4ruby/pull/129))
+
 ## 2.6.0 (25/10/2017)
 
 Features

data/Rakefile

@@ -8,13 +8,11 @@ namespace :test do
   desc "Test the basics of the adapter"
   Rake::TestTask.new(:units) do |t|
     t.test_files = FileList["test/unit/*_test.rb", "specs/**/*_spec.rb"]
-    t.verbose = true
   end
 
   desc "Run a test against the live API"
   Rake::TestTask.new(:integration) do |t|
     t.test_files = FileList["test/integration/*_test.rb"]
-    t.verbose = true
   end
 
 end

data/lib/raygun/client.rb

@@ -16,6 +16,7 @@ module Raygun
 
       enable_http_proxy if Raygun.configuration.proxy_settings[:address]
       self.class.base_uri Raygun.configuration.api_url
+      self.class.default_timeout(Raygun.configuration.error_report_send_timeout)
     end
 
     def require_api_key
@@ -242,7 +243,13 @@ module Raygun
       def create_entry(payload_hash)
         Raygun.log('sending payload to api')
 
-        self.class.post("/entries", verify_peer: true, verify: true, headers: @headers, body: JSON.generate(payload_hash))
+        self.class.post(
+          "/entries",
+          verify_peer: true,
+          verify: true,
+          headers: @headers,
+          body: JSON.generate(payload_hash),
+        )
       end
 
       def filter_params_with_blacklist(params_hash = {}, extra_filter_keys = nil)

data/lib/raygun/configuration.rb

@@ -77,6 +77,9 @@ module Raygun
     # Should the exceptions to Raygun be sent asynchronously?
     config_option :send_in_background
 
+    # How long to wait for the POST request to the API server before timing out
+    config_option :error_report_send_timeout
+
     # Exception classes to ignore by default
     IGNORE_DEFAULT = ['ActiveRecord::RecordNotFound',
                       'ActionController::RoutingError',
@@ -129,7 +132,8 @@ module Raygun
         api_url:                       'https://api.raygun.io/',
         breadcrumb_level:              :info,
         record_raw_data:               false,
-        send_in_background:            false
+        send_in_background:            false,
+        error_report_send_timeout:     10
       )
     end
 

data/lib/raygun/version.rb

@@ -1,3 +1,3 @@
 module Raygun
-  VERSION = "2.6.0"
+  VERSION = "2.7.0"
 end

data/lib/raygun.rb

@@ -51,6 +51,10 @@ module Raygun
       configuration.defaults
     end
 
+    def reset_configuration
+      @configuration = Configuration.new
+    end
+
     def configured?
       !!configuration.api_key
     end
@@ -118,7 +122,7 @@ module Raygun
     def track_exception_async(*args)
       future = Concurrent::Future.execute { track_exception_sync(*args) }
       future.add_observer(lambda do |_, value, reason|
-        if value == nil || value.response.code != "202"
+        if value == nil || !value.responds_to?(:response) || value.response.code != "202"
           log("unexpected response from Raygun, could indicate error: #{value.inspect}")
         end
       end, :call)
@@ -127,7 +131,10 @@ module Raygun
     def track_exception_sync(exception_instance, env, user, retry_count)
       if should_report?(exception_instance)
         log('attempting to send exception')
-        Client.new.track_exception(exception_instance, env, user)
+        resp = Client.new.track_exception(exception_instance, env, user)
+        log('sent payload to api')
+
+        resp
       end
     rescue Exception => e
       log('error sending exception to raygun, see failsafe logger for more information')

data/raygun4ruby.gemspec

@@ -25,8 +25,7 @@ Gem::Specification.new do |spec|
   spec.add_runtime_dependency "concurrent-ruby"
 
   spec.add_development_dependency "bundler", ">= 1.1"
-  spec.add_development_dependency "rake", "0.9.6"
-  spec.add_development_dependency "fakeweb", ["~> 1.3"]
+  spec.add_development_dependency "rake", "~> 11"
   spec.add_development_dependency "timecop"
   spec.add_development_dependency "minitest", "~> 4.2"
   spec.add_development_dependency "redis-namespace", ">= 1.3.1"
@@ -34,4 +33,5 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "sidekiq", [">= 3", "< 3.2.2"]
   spec.add_development_dependency "mocha"
   spec.add_development_dependency "pry"
+  spec.add_development_dependency "webmock"
 end

data/specs/raygun/raygun_spec.rb

@@ -0,0 +1,50 @@
+require_relative '../spec_helper'
+require 'stringio'
+
+describe Raygun do
+  let(:failsafe_logger) { FakeLogger.new }
+
+  describe '#track_exception' do
+    context 'send in background' do
+      before do
+        Raygun.setup do |c|
+          c.send_in_background = true
+          c.api_url = 'http://example.api'
+          c.api_key = 'foo'
+          c.debug = true
+          c.failsafe_logger = failsafe_logger
+        end
+      end
+
+      context 'request times out' do
+        before do
+          stub_request(:post, 'http://example.api/entries').to_timeout
+        end
+
+        it 'logs the failure to the failsafe logger' do
+          error = StandardError.new
+
+          Raygun.track_exception(error)
+
+          # Occasionally doesn't write to the failsafe logger, add small timeout to add some safety
+          sleep 0.1
+          failsafe_logger.get.must_match /Problem reporting exception to Raygun/
+        end
+      end
+    end
+  end
+
+  describe '#reset_configuration' do
+    it 'clears any customized configuration options' do
+      Raygun.setup do |c|
+        c.api_url = 'http://test.api'
+      end
+
+      Raygun.configuration.api_url.must_equal 'http://test.api'
+
+      Raygun.reset_configuration
+
+      Raygun.configuration.api_url.must_equal Raygun.default_configuration.api_url
+    end
+  end
+end

data/specs/spec_helper.rb

@@ -2,8 +2,28 @@ ENV['RACK_ENV'] = 'test'
 require_relative "../lib/raygun.rb"
 require "minitest/autorun"
 require "minitest/pride"
-require "fakeweb"
 require "timecop"
 require "mocha/mini_test"
+require "webmock/minitest"
+require 'stringio'
 
 alias :context :describe
+
+class FakeLogger
+  def initialize
+    @logger = StringIO.new
+  end
+
+  def info(message)
+    @logger.write(message)
+  end
+
+  def reset
+    @logger.string = ""
+  end
+
+  def get
+    @logger.string
+  end
+end
+

data/test/test_helper.rb

@@ -2,7 +2,6 @@ ENV['RACK_ENV'] = 'test'
 require_relative "../lib/raygun.rb"
 require "minitest/autorun"
 require "minitest/pride"
-require "fakeweb"
 require "timecop"
 require "mocha/mini_test"
 require 'stringio'
@@ -47,20 +46,17 @@ end
 class Raygun::UnitTest < MiniTest::Unit::TestCase
 
   def setup
-    FakeWeb.allow_net_connect = false
     Raygun.configuration.api_key = "test api key"
   end
 
-  def fake_successful_entry
-    FakeWeb.register_uri(:post, "https://api.raygun.io/entries", body: "", status: 202)
-  end
-
   def teardown
-    FakeWeb.clean_registry
-    FakeWeb.allow_net_connect = true
     reset_configuration
   end
 
+  def fake_successful_entry
+    stub_request(:post, 'https://api.raygun.io/entries').to_return(status: 202)
+  end
+
   def reset_configuration
     Raygun.configuration = Raygun::Configuration.new
   end

data/test/unit/configuration_test.rb

@@ -12,6 +12,10 @@ class ConfigurationTest < Raygun::UnitTest
     end
   end
 
+  def teardown
+    Raygun.reset_configuration
+  end
+
   def test_setting_api_key_and_version
     assert_equal 9.9,              Raygun.configuration.version
     assert_equal "a test api key", Raygun.configuration.api_key
@@ -179,4 +183,12 @@ class ConfigurationTest < Raygun::UnitTest
   def test_send_in_background_default
     assert_equal false, Raygun.configuration.send_in_background
   end
+
+  def test_error_report_send_timeout_default
+    assert_equal 10, Raygun.configuration.error_report_send_timeout
+  end
+
+  def test_enable_reporting_default
+    assert_equal true, Raygun.configuration.enable_reporting
+  end
 end

data/test/unit/resque_failure_test.rb

@@ -7,6 +7,8 @@ class ResqueFailureTest < Raygun::UnitTest
 
   def setup
     super
+
+    stub_request(:post, 'https://api.raygun.io/entries').to_return(status: 202)
     fake_successful_entry
   end
 
@@ -19,4 +21,4 @@ class ResqueFailureTest < Raygun::UnitTest
     ).save.success?
   end
 
-end
+end

data/test/unit/sidekiq_failure_test.rb

@@ -13,6 +13,8 @@ class SidekiqFailureTest < Raygun::UnitTest
 
   def setup
     super
+
+    stub_request(:post, 'https://api.raygun.io/entries').to_return(status: 202)
     fake_successful_entry
   end
 
@@ -27,4 +29,4 @@ class SidekiqFailureTest < Raygun::UnitTest
     assert Sidekiq.error_handlers.include?(Raygun::SidekiqReporter)
   end
 
-end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: raygun4ruby
 version: !ruby/object:Gem::Version
-  version: 2.6.0
+  version: 2.7.0
 platform: ruby
 authors:
 - Mindscape
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-10-24 00:00:00.000000000 Z
+date: 2018-02-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty
@@ -83,32 +83,18 @@ dependencies:
         version: '1.1'
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - '='
-      - !ruby/object:Gem::Version
-        version: 0.9.6
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - '='
-      - !ruby/object:Gem::Version
-        version: 0.9.6
-- !ruby/object:Gem::Dependency
-  name: fakeweb
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: '11'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: '11'
 - !ruby/object:Gem::Dependency
   name: timecop
   requirement: !ruby/object:Gem::Requirement
@@ -213,6 +199,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: webmock
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Ruby Adapter for Raygun.io
 email:
 - hello@raygun.io
@@ -251,6 +251,7 @@ files:
 - raygun4ruby.gemspec
 - specs/raygun/breadcrumbs/breadcrumb_spec.rb
 - specs/raygun/breadcrumbs/store_spec.rb
+- specs/raygun/raygun_spec.rb
 - specs/services/apply_whitelist_filter_to_payload_spec.rb
 - specs/spec_helper.rb
 - test/integration/client_test.rb