raygun4ruby 2.1.0 → 2.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f4273509fb2b19bd65ee19cb8589607b51938cde
-  data.tar.gz: 592cbaa95d8afc226508600af62f53c8086201f1
+  metadata.gz: 7f6e237c42f9f818bec75c11a19c4122cc7dbbcd
+  data.tar.gz: fe2879c2c31a4138ff3f38f41add405698bcb6c9
 SHA512:
-  metadata.gz: 5e1fca4593ff9aa4d19de3b174750ec2142f8cfc599b2cc5358b9dcb8e3eb73930e905eb71dd27d546c984d6525b243f249a2b8f4c1d3b5bdedea387555d3adf
-  data.tar.gz: 4a7249e4ed86910f9d66a43fbb7937bed5e3b649af2a880c49c7a9663b2ddef63ddcf8c9416b712930e2cac7b4257b28b2dc92e36258f94718b307c6c3915e7b
+  metadata.gz: 41887a583b9417a02908eb9485df7d276d9c7be4e0bcaa4c2f1c77c3c4a439f004d51f254a6935fc0425399b6cac97c29bde3e295087fa9e6ee0b40e8167dabf
+  data.tar.gz: 4a15d7e5d04857888ad998795a32a55d206bcb76a6c888f4a68d70f693eeaee1e37da3bcd9996e3bc381c2d7747a31be0b71f16e60d7720c357c9b5b059920d5

data/CHANGELOG.md

@@ -1,3 +1,11 @@
+## 2.2.0
+
+Features
+  - Opt in support for sending exceptions in a background thread to not block web request thread during IO ([#117](https://github.com/MindscapeHQ/raygun4ruby/pull/117))
+
+Bugfixes
+ - Don't attempt to read raw data during GET requests or if rack.input buffer is empty
+
 ## 2.1.0
 
 Features

data/README.md

@@ -33,7 +33,7 @@ You can then test your Raygun integration by running:
 
 You should see an "ItWorksException" appear in your Raygun dashboard. You're ready to zap those errors!
 
-NB: Raygun4Ruby currently requires Ruby >= 1.9
+NB: Raygun4Ruby currently requires Ruby >= 2.0
 
 Note that the generator will create a file in `config/initializers` called "raygun.rb". If you need to do any further configuration or customization of Raygun, that's the place to do it!
 

data/lib/raygun.rb

@@ -1,3 +1,4 @@
+require "concurrent"
 require "httparty"
 require "logger"
 require "json"
@@ -54,25 +55,10 @@ module Raygun
     end
 
     def track_exception(exception_instance, env = {}, user = nil, retry_count = 1)
-      if should_report?(exception_instance)
-        log("[Raygun] Tracking Exception...")
-        Client.new.track_exception(exception_instance, env, user)
-      end
-    rescue Exception => e
-      if configuration.failsafe_logger
-        failsafe_log("Problem reporting exception to Raygun: #{e.class}: #{e.message}\n\n#{e.backtrace.join("\n")}")
-      end
-
-      if retry_count > 0
-        new_exception = e.exception("raygun4ruby encountered an exception processing your exception")
-        new_exception.set_backtrace(e.backtrace)
-
-        env[:custom_data] ||= {}
-        env[:custom_data].merge!(original_stacktrace: exception_instance.backtrace)
-
-        track_exception(new_exception, env, user, retry_count - 1)
+      if configuration.send_in_background
+        track_exception_async(exception_instance, env, user, retry_count)
       else
-        raise e
+        track_exception_sync(exception_instance, env, user, retry_count)
       end
     end
 
@@ -122,6 +108,39 @@ module Raygun
 
     private
 
+    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"
+          log("[Raygun] unexpected response from Raygun, could indicate error: #{value.inspect}")
+        end
+      end, :call)
+    end
+
+    def track_exception_sync(exception_instance, env, user, retry_count)
+      if should_report?(exception_instance)
+        log("[Raygun] Tracking Exception...")
+        Client.new.track_exception(exception_instance, env, user)
+      end
+    rescue Exception => e
+      if configuration.failsafe_logger
+        failsafe_log("Problem reporting exception to Raygun: #{e.class}: #{e.message}\n\n#{e.backtrace.join("\n")}")
+      end
+
+      if retry_count > 0
+        new_exception = e.exception("raygun4ruby encountered an exception processing your exception")
+        new_exception.set_backtrace(e.backtrace)
+
+        env[:custom_data] ||= {}
+        env[:custom_data].merge!(original_stacktrace: exception_instance.backtrace)
+
+        track_exception(new_exception, env, user, retry_count - 1)
+      else
+        raise e
+      end
+    end
+
+
     def print_api_key_warning
       $stderr.puts(NO_API_KEY_MESSAGE)
     end

data/lib/raygun/client.rb

@@ -136,8 +136,11 @@ module Raygun
 
         request = Rack::Request.new(rack_env)
         input = rack_env['rack.input']
+        return if request.get?
 
-        if input && !request.form_data?
+        # If size is 0 the buffer is at best empty and at worst
+        # something like the Puma::NullIO buffer which is missing methods
+        if input && input.size && !request.form_data?
           current_position = input.pos
           input.rewind
 

data/lib/raygun/configuration.rb

@@ -74,6 +74,9 @@ module Raygun
     # form submissions and will not be filtered by the blacklist
     config_option :record_raw_data
 
+    # Should the exceptions to Raygun be sent asynchronously?
+    config_option :send_in_background
+
     # Exception classes to ignore by default
     IGNORE_DEFAULT = ['ActiveRecord::RecordNotFound',
                       'ActionController::RoutingError',
@@ -111,7 +114,7 @@ module Raygun
       @config_values = {}
 
       # set default attribute values
-      @defaults = OpenStruct.new({
+      @defaults = OpenStruct.new(
         ignore:                        IGNORE_DEFAULT,
         custom_data:                   {},
         tags:                          [],
@@ -125,8 +128,9 @@ module Raygun
         debug:                         false,
         api_url:                       'https://api.raygun.io/',
         breadcrumb_level:              :info,
-        record_raw_data:               false
-      })
+        record_raw_data:               false,
+        send_in_background:            false
+      )
     end
 
     def [](key)

data/lib/raygun/version.rb

@@ -1,3 +1,3 @@
 module Raygun
-  VERSION = "2.1.0"
+  VERSION = "2.2.0"
 end

data/raygun4ruby.gemspec

@@ -22,6 +22,7 @@ Gem::Specification.new do |spec|
   spec.add_runtime_dependency "httparty", "> 0.13.7"
   spec.add_runtime_dependency "json"
   spec.add_runtime_dependency "rack"
+  spec.add_runtime_dependency "concurrent-ruby"
 
   spec.add_development_dependency "bundler", ">= 1.1"
   spec.add_development_dependency "rake", "0.9.6"

data/test/unit/client_test.rb

@@ -213,7 +213,7 @@ class ClientTest < Raygun::UnitTest
       queryString: { "a" => "b", "c" => "4945438" },
       headers:     { "Version"=>"HTTP/1.1", "Host"=>"localhost:3000", "Cookie"=>"cookieval" },
       form:        {},
-      rawData:     {}
+      rawData:     nil
     }
 
     assert_equal expected_hash, @client.send(:request_information, env_hash)
@@ -550,7 +550,7 @@ class ClientTest < Raygun::UnitTest
       queryString: { "a" => "b", "c" => "4945438" },
       headers:     { "Version"=>"HTTP/1.1", "Host"=>"localhost:3000", "Cookie"=>"cookieval" },
       form:        {},
-      rawData:     {}
+      rawData:     nil
     }
 
     details = @client.send(:build_payload_hash, test_exception, env_hash)[:details]
@@ -574,7 +574,7 @@ class ClientTest < Raygun::UnitTest
       queryString: "[FILTERED]",
       headers:     { "Version"=>"HTTP/1.1", "Host"=>"localhost:3000", "Cookie"=>"cookieval" },
       form:        {},
-      rawData:     {}
+      rawData:     nil
     }
 
     details = @client.send(:build_payload_hash, test_exception, sample_env_hash)[:details]
@@ -597,7 +597,7 @@ class ClientTest < Raygun::UnitTest
       queryString: { "a" => "b", "c" => "4945438" },
       headers:     { "Version"=>"HTTP/1.1", "Host"=>"localhost:3000", "Cookie"=>"cookieval" },
       form:        {},
-      rawData:     {}
+      rawData:     nil
     }
 
     details = @client.send(:build_payload_hash, test_exception, env_hash)[:details]

data/test/unit/configuration_test.rb

@@ -159,4 +159,8 @@ class ConfigurationTest < Raygun::UnitTest
   def test_record_raw_data_default
     assert_equal false, Raygun.configuration.record_raw_data
   end
+
+  def test_send_in_background_default
+    assert_equal false, Raygun.configuration.send_in_background
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: raygun4ruby
 version: !ruby/object:Gem::Version
-  version: 2.1.0
+  version: 2.2.0
 platform: ruby
 authors:
 - Mindscape
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-04-27 00:00:00.000000000 Z
+date: 2017-05-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty
@@ -53,6 +53,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: concurrent-ruby
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement