sentry-raven 2.4.0 → 2.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: cb65022aed8ddc08bcb526c2c71d0b53b6cda940
-  data.tar.gz: '079e7ee3652a38722ebf913095cfd45be86d79d7'
+  metadata.gz: fb8e99d6cd40b4a9bcebae0aa4c2b6a9ae92c65f
+  data.tar.gz: f0b34e2f0061b92c0b94dbfce8409b415850a756
 SHA512:
-  metadata.gz: 4aafa4ea87fc78346f7050cddfc71f445e6a9deccc5990c1cebfaf8b62882178a8160d40bf9267ec014c826ffa8a1a2bc8b3cea13a6f748ccca6b2c4feda3e5b
-  data.tar.gz: 00000f1fe367791b055f763e3100223bc3f71827a41d60c2dcfcbe63baf4ec25214e3c3a21383901c56e5d4c4c0d72b6be07b1534cf0ecc6b5c5fe1eb0f3bb13
+  metadata.gz: 89cd1bda236dfc17bde728a9b69ddcc0d2f4bbffdd2a8697969662c977addf14f180285132987b20cfa53f9398cf3b80309bc781bd502700ce6a4b20a663ccde
+  data.tar.gz: 11d7601a11bd66e3881f457f60c9db1bb253a4a93fa35b32bb61447d938f74019b7fa219a3f4cf8d797cac48835e8d795a2417f5b9b4d793a97596203516286c

data/README.md

@@ -143,7 +143,7 @@ For more information, see [Context](https://docs.sentry.io/clients/ruby/context/
 ## More Information
 
 * [Documentation](https://docs.getsentry.com/hosted/clients/ruby/)
-* [Bug Tracker](https://github.com/getsentry/raven-ruby/issues>)
-* [Code](https://github.com/getsentry/raven-ruby>)
-* [Mailing List](https://groups.google.com/group/getsentry>)
+* [Bug Tracker](https://github.com/getsentry/raven-ruby/issues)
+* [Code](https://github.com/getsentry/raven-ruby)
+* [Mailing List](https://groups.google.com/group/getsentry)
 * [IRC](irc://irc.freenode.net/sentry>)  (irc.freenode.net, #sentry)

data/lib/raven/client.rb

@@ -64,7 +64,7 @@ module Raven
 
     def encode(event)
       hash = @processors.reduce(event.to_hash) { |a, e| e.process(a) }
-      encoded = JSON.generate(hash)
+      encoded = JSON.fast_generate(hash)
 
       case configuration.encoding
       when 'gzip'

data/lib/raven/integrations/sidekiq.rb

@@ -51,7 +51,7 @@ module Raven
     # this will change in the future:
     # https://github.com/mperham/sidekiq/pull/3161
     def culprit_from_context(context)
-      classname = (context["class"] || (context["job"] && context["job"]["class"]))
+      classname = (context["class"] || (context["job"] && (context["job"]["wrapped"] || context["job"]["class"])))
       if classname
         "Sidekiq/#{classname}"
       elsif context["event"]

data/lib/raven/interfaces/stack_trace.rb

@@ -37,7 +37,7 @@ module Raven
       end
 
       def filename
-        return nil if abs_path.nil?
+        return if abs_path.nil?
 
         prefix =
           if under_project_root? && in_app
@@ -60,7 +60,7 @@ module Raven
       end
 
       def longest_load_path
-        $LOAD_PATH.select { |s| abs_path.start_with?(s.to_s) }.sort_by { |s| s.to_s.length }.last
+        $LOAD_PATH.select { |s| abs_path.start_with?(s) }.max_by(&:length)
       end
 
       def to_hash(*args)

data/lib/raven/processor/removecircularreferences.rb

@@ -1,14 +1,16 @@
 module Raven
   class Processor::RemoveCircularReferences < Processor
-    def process(v, visited = [])
-      return "(...)" if visited.include?(v.__id__)
-      visited += [v.__id__]
-      if v.is_a?(Hash)
-        v.each_with_object({}) { |(k, v_), memo| memo[k] = process(v_, visited) }
-      elsif v.is_a?(Array)
-        v.map { |v_| process(v_, visited) }
+    def process(value, visited = [])
+      return "(...)" if visited.include?(value.__id__)
+      visited << value.__id__ if value.is_a?(Array) || value.is_a?(Hash)
+
+      case value
+      when Hash
+        !value.frozen? ? value.merge!(value) { |_, v| process v, visited } : value.merge(value) { |_, v| process v, visited }
+      when Array
+        !value.frozen? ? value.map! { |v| process v, visited } : value.map { |v| process v, visited }
       else
-        v
+        value
       end
     end
   end

data/lib/raven/processor/sanitizedata.rb

@@ -14,31 +14,27 @@ module Raven
       self.sanitize_credit_cards = client.configuration.sanitize_credit_cards
     end
 
-    def process(value)
-      return value if value.frozen?
-      value.each_with_object(value) { |(k, v), memo| memo[k] = sanitize(k, v) }
-    end
-
-    def sanitize(k, v)
-      if v.is_a?(Hash)
-        process(v)
-      elsif v.is_a?(Array)
-        v.map { |a| sanitize(k, a) }
-      elsif k.to_s == 'query_string'
-        sanitize_query_string(v)
-      elsif v.is_a?(Integer) && matches_regexes?(k, v)
-        INT_MASK
-      elsif v.is_a?(String)
-        if fields_re.match(v.to_s) && (json = parse_json_or_nil(v))
+    def process(value, key = nil)
+      case value
+      when Hash
+        !value.frozen? ? value.merge!(value) { |k, v| process v, k } : value.merge(value) { |k, v| process v, k }
+      when Array
+        !value.frozen? ? value.map! { |v| process v, key } : value.map { |v| process v, key }
+      when Integer
+        matches_regexes?(key, value.to_s) ? INT_MASK : value
+      when String
+        if value =~ fields_re && (json = parse_json_or_nil(value))
           # if this string is actually a json obj, convert and sanitize
-          json.is_a?(Hash) ? process(json).to_json : v
-        elsif matches_regexes?(k, v)
+          process(json).to_json
+        elsif matches_regexes?(key, value)
           STRING_MASK
+        elsif key == 'query_string' || key == :query_string
+          sanitize_query_string(value)
         else
-          v
+          value
         end
       else
-        v
+        value
       end
     end
 
@@ -51,8 +47,8 @@ module Raven
     end
 
     def matches_regexes?(k, v)
-      (sanitize_credit_cards && CREDIT_CARD_RE.match(v.to_s)) ||
-        fields_re.match(k.to_s)
+      (sanitize_credit_cards && v =~ CREDIT_CARD_RE) ||
+        k =~ fields_re
     end
 
     def fields_re
@@ -70,6 +66,7 @@ module Raven
     end
 
     def parse_json_or_nil(string)
+      return unless string.start_with?("[", "{")
       JSON.parse(string)
     rescue JSON::ParserError, NoMethodError
       nil

data/lib/raven/processor/utf8conversion.rb

@@ -1,29 +1,28 @@
 module Raven
   class Processor::UTF8Conversion < Processor
     def process(value)
-      if value.is_a? Array
-        value.map { |v| process v }
-      elsif value.is_a? Hash
-        value.merge(value) { |_, v| process v }
-      elsif value.is_a?(Exception) && !value.message.valid_encoding?
+      case value
+      when Array
+        !value.frozen? ? value.map! { |v| process v } : value.map { |v| process v }
+      when Hash
+        !value.frozen? ? value.merge!(value) { |_, v| process v } : value.merge(value) { |_, v| process v }
+      when Exception
+        return value if value.message.valid_encoding?
         clean_exc = value.class.new(clean_invalid_utf8_bytes(value.message))
         clean_exc.set_backtrace(value.backtrace)
         clean_exc
-      else
+      when String
+        return value if value.valid_encoding?
         clean_invalid_utf8_bytes(value)
+      else
+        value
       end
     end
 
     private
 
     def clean_invalid_utf8_bytes(obj)
-      if obj.respond_to?(:to_utf8)
-        obj.to_utf8
-      elsif obj.respond_to?(:encoding) && obj.is_a?(String)
-        obj.encode('UTF-16', :invalid => :replace, :undef => :replace, :replace => '').encode('UTF-8')
-      else
-        obj
-      end
+      obj.encode('UTF-16', :invalid => :replace, :undef => :replace, :replace => '').encode('UTF-8')
     end
   end
 end

data/lib/raven/transports/http.rb

@@ -27,7 +27,7 @@ module Raven
           req.headers['X-Sentry-Auth'] = auth_header
           req.body = data
         end
-      rescue Faraday::ClientError => ex
+      rescue Faraday::Error => ex
         error_info = ex.message
         if ex.response && ex.response[:headers]['x-sentry-error']
           error_info += " Error in headers is: #{ex.response[:headers]['x-sentry-error']}"

data/lib/raven/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 module Raven
   # Freezing this constant breaks in 1.9.x
-  VERSION = "2.4.0" # rubocop:disable Style/MutableConstant
+  VERSION = "2.5.0" # rubocop:disable Style/MutableConstant
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sentry-raven
 version: !ruby/object:Gem::Version
-  version: 2.4.0
+  version: 2.5.0
 platform: ruby
 authors:
 - Sentry Team
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-03-07 00:00:00.000000000 Z
+date: 2017-05-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
@@ -193,7 +193,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.8
+rubygems_version: 2.6.11
 signing_key: 
 specification_version: 4
 summary: A gem that provides a client interface for the Sentry error logger