action_reporter 1.0.2 → 1.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '0569ae232aed54ea4777b366d95fef80a4058d02a2c22bccdf1a56614282ff99'
-  data.tar.gz: 22317e6ea26f6a06f1626a7812ed65dbee279b9b45f6485166cb0c2bc0da7bf1
+  metadata.gz: 49736906f57c97cd86a7f0963fa78a63d6cbbb9d0fe03d601b987b376dca8e00
+  data.tar.gz: 6446a9feb95c216715b07b1775239b600a2ad1ae8083f7fbacdf86cd97b44469
 SHA512:
-  metadata.gz: 34c9ca0d8dfed3925742e698a2f013aebdda61bbad8f6a63e5751e42cb56c57d605b67965fa612dee07e2d84d467fd749abe3d460ad4ad9a921568817f9bfdb8
-  data.tar.gz: 00b99d87c2ece2ba07779ab176b063e7ef862466f4f7354905da3272b8a7b622b7775f7e92cb144615c4759667be7d98ae2f704cbcfc94c5c198f22bda3ac6e3
+  metadata.gz: 7294c392d117b9a54625bb80b417c489408e2068d33b9338ec8d3bc60a0a91f3e360163fdcc9e6830eba1231a35ec5fe49f9c58b089f60e983e841302f80ede1
+  data.tar.gz: 047d3ee881ea607458bd29d97dfcb72e3aee38b899aaa2893532a38ccf1f9c466ffd58b4459a0d70355e895a722419ac11cacca60e6f213c3b9cdbe978541715

data/CHANGELOG.md

@@ -1,3 +1,11 @@
+# 1.0.4
+
+* Move `transform_context` to individual reporter classes
+
+# 1.0.3
+
+* Fix scoutapm notice method
+
 # 1.0.2
 
 * Add ruby version support for versions lower than 3.2.0

data/action_reporter.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name = "action_reporter"
-  s.version = "1.0.2"
+  s.version = "1.0.4"
 
   s.license = "MIT"
 

data/lib/action_reporter/audited_reporter.rb

@@ -2,10 +2,6 @@ module ActionReporter
   class AuditedReporter < Base
     class_accessor "Audited"
 
-    def transform_context?
-      false
-    end
-
     def notify(*)
     end
 
@@ -13,9 +9,6 @@ module ActionReporter
       Audited.store[:current_remote_address] = args[:remote_addr] if args[
         :remote_addr
       ].present?
-      Audited.store[:audited_user] = args[:audited_user] if args[
-        :audited_user
-      ].present?
     end
 
     def reset_context
@@ -26,5 +19,9 @@ module ActionReporter
     def audited_user
       Audited.store[:audited_user]
     end
+
+    def audited_user=(user)
+      Audited.store[:audited_user] = user
+    end
   end
 end

data/lib/action_reporter/base.rb

@@ -7,8 +7,14 @@ module ActionReporter
       const_set(const_name, Object.const_get(class_name))
     end
 
-    def transform_context?
-      true
+    def transform_context(context)
+      ActionReporter::Utils.deep_transform_values(context) do |value|
+        if value.respond_to?(:to_global_id)
+          value.to_global_id.to_s
+        else
+          value
+        end
+      end
     end
 
     def notify(*)

data/lib/action_reporter/honeybadger_reporter.rb

@@ -3,11 +3,13 @@ module ActionReporter
     class_accessor "Honeybadger"
 
     def notify(error, context: {})
-      Honeybadger.notify(error, context: context)
+      new_context = transform_context(context)
+      Honeybadger.notify(error, context: new_context)
     end
 
     def context(args)
-      Honeybadger.context(args)
+      new_context = transform_context(args)
+      Honeybadger.context(new_context)
     end
 
     def reset_context

data/lib/action_reporter/rails_reporter.rb

@@ -3,16 +3,15 @@ module ActionReporter
     class_accessor "Rails"
 
     def notify(error, context: {})
+      new_context = transform_context(context)
       Rails.logger.info(
-        "Reporter notification: #{error.inspect}, #{context.inspect}"
+        "Reporter notification: #{error.inspect}, #{new_context.inspect}"
       )
     end
 
     def context(args)
-      Rails.logger.info("Reporter context: #{args.inspect}")
-    end
-
-    def reset_context
+      new_context = transform_context(args)
+      Rails.logger.info("Reporter context: #{new_context.inspect}")
     end
   end
 end

data/lib/action_reporter/scout_apm_reporter.rb

@@ -1,14 +1,16 @@
 module ActionReporter
   class ScoutApmReporter < Base
-    class_accessor "ScoutApm::Agent"
+    class_accessor "ScoutApm::Error"
     class_accessor "ScoutApm::Context"
 
-    def notify(*args, **kwargs)
-      ScoutApmAgent.instance.error(*args, **kwargs)
+    def notify(error, context: {})
+      self.context(context)
+      ScoutApmError.capture(error)
     end
 
     def context(args)
-      ScoutApmContext.add(args)
+      new_context = transform_context(args)
+      ScoutApmContext.add(new_context)
     end
   end
 end

data/lib/action_reporter/sentry_reporter.rb

@@ -8,13 +8,12 @@ module ActionReporter
     end
 
     def context(args)
-      args.each do |key, value|
-        Sentry.configure_scope { |scope| scope.set_context(key, value) }
-      end
+      new_context = transform_context(args)
+      Sentry.set_context(args)
     end
 
-    def reset_context
-      Sentry.configure_scope { |scope| scope.clear_breadcrumbs }
+    def audited_user=(user)
+      Sentry.set_user(global_id: user.to_global_id.to_s) if user
     end
   end
 end

data/lib/action_reporter/version.rb

@@ -1,3 +1,3 @@
 module ActionReporter
-  VERSION = '1.0.2'
+  VERSION = '1.0.4'
 end

data/lib/action_reporter.rb

@@ -30,35 +30,37 @@ module ActionReporter
 
   def notify(error, context: {})
     enabled_reporters.each do |reporter|
-      new_context =
-        reporter.transform_context? ? transform_context(context) : context
-      reporter.notify(error, context: new_context)
+      next unless reporter.respond_to?(:notify)
+
+      reporter.notify(error, context: context)
     end
   end
 
   def context(args)
     enabled_reporters.each do |reporter|
-      new_args = reporter.transform_context? ? transform_context(args) : args
-      reporter.context(new_args)
+      next unless reporter.respond_to?(:context)
+
+      reporter.context(args)
     end
   end
 
   def reset_context
-    enabled_reporters.each(&:reset_context)
+    enabled_reporters.each do |reporter|
+      next unless reporter.respond_to?(:reset_context)
+
+      reporter.reset_context
+    end
   end
 
   def audited_user
     enabled_reporters.find { |r| r.respond_to?(:audited_user) }&.audited_user
   end
 
-  def transform_context(context)
-    filtered_context = context.reject { |k, _| k == :audited_user }
-    ActionReporter::Utils.deep_transform_values(filtered_context) do |value|
-      if value.respond_to?(:to_global_id)
-        value.to_global_id.to_s
-      else
-        value
-      end
+  def audited_user=(user)
+    enabled_reporters.each do |reporter|
+      next unless reporter.respond_to?(:audited_user=)
+
+      reporter.audited_user = user
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: action_reporter
 version: !ruby/object:Gem::Version
-  version: 1.0.2
+  version: 1.0.4
 platform: ruby
 authors:
 - Andrei Makarov