upfluence-utils 0.11.4 → 0.12.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b5084c4d4ec0053296da08af0f701e3533c3c04e097c58bd48d674ab6ce02f15
-  data.tar.gz: 04dff5174d8d510c22c8236f54d1033100bdde9102955e66bb28e25e8c94146e
+  metadata.gz: f2c9dcbd049e1e8c27d2ec9f7075cccacb7f4814d23ea04def031a0f9f00957f
+  data.tar.gz: 1f6b5c5812beaccfbf78123004f503f9135c96aedef1097cede9df3f007c6e92
 SHA512:
-  metadata.gz: 10db3f2f7d9e1ae72d51ae9f6007d994b6e1b7d86752a68c94d212c1bb894360c6bf7f7f88857afb67c8fe780077f72d251499b0146d32583aa4686f4a24568f
-  data.tar.gz: 811cba7d199587b6d65ebac7a613c23e47f159c5e581d1d09340c34c76a684a8b1d1ae6a56f1a97eae93dabc717673246d8a65805aa35d32fd94834da2fe1cd5
+  metadata.gz: 7c9e85e446a91a43b0d71d855a3b5c8c601f84f77582b6f3f32c725eadc96682bed636d6987dbab210b98b64c437a92412df89e6ef4494a6b1b01d49e9aa4a79
+  data.tar.gz: 10f73661eccb80401be50c860b660b0be5707f60b9d864bbe6e49cd1e0e93a91ee1dc1fbcbdf99adb0f59936dcdd2115424657edb76d8345ab46c1dd95811232

data/lib/upfluence/error_logger/null.rb

@@ -18,7 +18,7 @@ module Upfluence
         end
       end
 
-      def notify(error, _method, *_args)
+      def notify(error, *_args)
         Upfluence.logger.error(error.inspect)
       end
 

data/lib/upfluence/error_logger/sentry.rb

@@ -1,66 +1,116 @@
-require 'raven'
+require 'sentry-ruby'
 
 module Upfluence
   module ErrorLogger
     class Sentry
-      EXCLUDED_ERRORS = (Raven::Configuration::IGNORE_DEFAULT + ['Identity::Thrift::Forbidden'])
+      EXCLUDED_ERRORS = (::Sentry::Configuration::IGNORE_DEFAULT + ['Identity::Thrift::Forbidden'])
+      MAX_TAG_SIZE = 8 * 1024
 
       def initialize
-        ::Raven.configure do |config|
-          config.dsn = ENV['SENTRY_DSN']
-          config.current_environment = Upfluence.env
+        @tag_extractors = []
+
+        ::Sentry.init do |config|
+          config.send_default_pii = true
+          config.dsn = ENV.fetch('SENTRY_DSN', nil)
+          config.environment = Upfluence.env
           config.excluded_exceptions = EXCLUDED_ERRORS
           config.logger = Upfluence.logger
-          config.release = "#{ENV['PROJECT_NAME']}-#{ENV['SEMVER_VERSION']}"
-          config.tags = {
-            unit_name: unit_name,
-            unit_type: unit_type
-          }.select { |_, v| !v.nil? }
+          config.release = "#{ENV.fetch('PROJECT_NAME', nil)}-#{ENV.fetch('SEMVER_VERSION', nil)}"
+          config.enable_tracing = false
+          config.auto_session_tracking = false
+        end
+
+        ::Sentry.set_tags(
+          { unit_name: unit_name, unit_type: unit_type }.select { |_, v| v }
+        )
+
+        ::Sentry.with_scope do |scope|
+          scope.add_event_processor do |event, hint|
+            tags = @tag_extractors.map(&:extract).compact.reduce({}, &:merge)
+
+            exc = hint[:exception]
+
+            tags.merge!(exc.tags) if exc.respond_to? :tags
+
+            tx_name = transaction_name(tags)
+
+            event.transaction = tx_name if tx_name
+            event.extra.merge!(prepare_extra(tags))
+
+            event
+          end
         end
       end
 
-      def notify(error, method, *args)
-        begin
-          Raven.capture_exception(
-            error,
-            extra: { method: method, arguments: args.map(&:inspect) },
-            tags: { method: method }
-          )
-        rescue Raven::Error => e
-          Upfluence.logger.error e.message
+      def append_tag_extractors(klass)
+        @tag_extractors << klass
+      end
+
+      def notify(error, *args)
+        ::Sentry.with_scope do |scope|
+          context = args.reduce({}) do |acc, arg|
+            v = if arg.is_a?(Hash)
+                  arg
+                else
+                  key = acc.empty? ? 'method' : "arg_#{acc.length}"
+                  { key => arg.inspect }
+                end
+
+            acc.merge(v)
+          end
+
+          scope.set_extras(prepare_extra(context))
+
+          ::Sentry.capture_exception(error)
         end
+      rescue ::Sentry::Error => e
+        Upfluence.logger.warning e.message
       end
 
       def user=(user)
-        Raven.user_context(id: user.id, email: user.email)
+        ::Sentry.set_user(id: user.id, email: user.email)
       end
 
       def middleware
-        ::Raven::Rack
+        ::Sentry::Rack::CaptureExceptions
       end
 
       def ignore_exception(*klss)
         klss.each do |kls|
-          puts kls
           case kls.class
           when Class
-            Raven.configuration.excluded_exceptions << kls.name
+            ::Sentry.configuration.excluded_exceptions << kls.name
           when String
-            Raven.configuration.excluded_exceptions << kls
+            ::Sentry.configuration.excluded_exceptions << kls
           else
-            Upfluence.logger.warn e.message
+            Upfluence.logger.warn "Unexcepted argument for ignore_exception #{kls}"
           end
         end
       end
 
       private
 
+      def prepare_extra(tags)
+        tags.select { |_k, v| v.respond_to?(:size) && v.size < MAX_TAG_SIZE }
+      end
+
+      def transaction_name(tags)
+        return tags['transaction'] if tags['transaction']
+
+        svc = tags['thrift.request.service']
+        mth = tags['thrift.request.method']
+
+        return "#{svc}##{mth}" if svc && mth
+
+        nil
+      end
+
       def unit_name
-        ENV['UNIT_NAME'].split('.').first if ENV['UNIT_NAME']
+        ENV['UNIT_NAME']&.split('.')&.first
       end
 
       def unit_type
-        unit_name.split('@').first if unit_name
+        unit_name&.split('@')&.first
       end
     end
   end

data/lib/upfluence/utils/version.rb

@@ -1,5 +1,5 @@
 module Upfluence
   module Utils
-    VERSION = '0.11.4'.freeze
+    VERSION = '0.12.0'.freeze
   end
 end

data/rbutils.gemspec

@@ -23,7 +23,7 @@ Gem::Specification.new do |spec|
   spec.add_runtime_dependency 'upfluence-thrift'
   spec.add_runtime_dependency 'sinatra'
   spec.add_runtime_dependency 'redis'
-  spec.add_runtime_dependency 'sentry-raven'
+  spec.add_runtime_dependency 'sentry-ruby'
   spec.add_runtime_dependency 'sinatra-contrib'
   spec.add_runtime_dependency 'activesupport'
   spec.add_runtime_dependency 'puma'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: upfluence-utils
 version: !ruby/object:Gem::Version
-  version: 0.11.4
+  version: 0.12.0
 platform: ruby
 authors:
 - Upfluence
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-06-05 00:00:00.000000000 Z
+date: 2023-06-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -95,7 +95,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: sentry-raven
+  name: sentry-ruby
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="