activeerror 1.0.7 → 1.0.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7ec417a0e0ca10408c0ccfdba79721e4e92c9e4ba64dafc888f4913f8c972493
-  data.tar.gz: b311b8891817daf40b3494c23ff957c2d161f7ce28a357f4a3f106fe83785b0b
+  metadata.gz: 7558b7d240e80821f7d6d8640b5ca6ef7ef5b34634f40e3482bfc99bf64107ea
+  data.tar.gz: e10bda1728e93ca0ecbd4051e406cbaeb17548599c6d98591b5b9083036cb371
 SHA512:
-  metadata.gz: 782e60a069cc4b71af4fd09cf43b679f4520dcf589f1451d9fd33936ec00821fab333267ec2e45de39f533a948a6c28a267974e1984ce831a16d6a30861c53d3
-  data.tar.gz: ac1a8ae7cafc8e838ead1441b8b92138bb2b0419ff895c373656568645e3e0656eceece8c9fcd6c0c76e37b6981850ce324bd03d325c60c1f843d74bc171fd52
+  metadata.gz: f50369819d20d1b2c36ab8b6572f02ffd33da5c2abac74f1ecd177dbb0283493d2b9d572c3265ba135e0ece593c55bd1374b361aed66dcc90d16b5122c0bc5fd
+  data.tar.gz: 76d577c8e82e6e95ef1cdc9c8e01c12789278ae1f21a4c5424a03a484985990251fc48b8ab520dd7f4e1139a5c66d1972d11ddd913aed98bd46e4a3954e8fb8b

data/README.md

@@ -8,8 +8,9 @@ looking at using your data as training
 data([link](https://blog.sentry.io/ai-privacy-and-terms-of-service-updates/?original_referrer=https%3A%2F%2Fsentry.io%2F))
 there should be an easy open source alternative where you control the data.
 
-ActiveError hooks into the middleware of Rails.
-These third party error loggers also try to make their own fancy
+ActiveError hooks into the [error reporting
+api](https://guides.rubyonrails.org/error_reporting.html) baked directly into
+Rails. These third party error loggers also try to make their own fancy
 backtrace and debugging view. But the one Rails developers are most comfortable
 with is the one baked into Rails that we use everyday in development. So with
 ActiveError, when an error gets raised it's captured and stored in the

data/db/migrate/20200727225318_create_active_error_instances.rb

@@ -4,7 +4,7 @@ class CreateActiveErrorInstances < ActiveRecord::Migration[7.1]
   def change
     create_table :active_error_instances do |t|
       t.belongs_to :fault
-      t.string :url
+      t.text :url
       t.binary :headers, limit: 512.megabytes
       t.binary :parameters, limit: 512.megabytes
 

data/lib/active_error/engine.rb

@@ -1,25 +1,19 @@
 # frozen_string_literal: true
 
 module ActiveError
-  class Engine < ::Rails::Engine
-    ActiveJobRequest = Struct.new(:parameters, :filtered_parameters, :env, :url)
+  ActiveJobRequest = Struct.new(:parameters, :filtered_parameters, :env, :url)
 
+  class Engine < ::Rails::Engine
     isolate_namespace ActiveError
 
-    initializer "active_error.middleware" do |app|
-      unless Rails.env.local?
-        app.config.middleware.use ActiveError::Middleware
+    initializer "active_error.middleware" do |_app|
+      ActionController::Base.before_action do
+        Rails.error.set_context(active_error_request: request)
+      end
 
-        ::ActiveJob::Base.class_eval do |base|
-          base.set_callback :perform, :around do |_param, block|
-            block.call
-          rescue StandardError => e
-            ActiveError::Captor.new(exception: e, request: ActiveJobRequest.new(
-              serialize, serialize
-            )).capture
-            raise e
-          end
-        end
+      ActiveJob::Base.before_perform do
+        Rails.error.set_context(active_error_request:
+          ActiveJobRequest.new(serialize, serialize))
       end
     end
   end

data/lib/active_error/railtie.rb

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+module ActiveError
+  class ErrorSubscriber
+    def report(exception, context:, **_opts)
+      return if Rails.env.local?
+
+      Captor.new(exception:, request: context[:active_error_request]).capture
+    end
+  end
+
+  class Railtie < ::Rails::Railtie
+    initializer "active_error.error_reporter" do |_app|
+      Rails.error.subscribe(::ActiveError::ErrorSubscriber.new)
+    end
+  end
+end

data/lib/active_error/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ActiveError
-  VERSION = "1.0.7"
+  VERSION = "1.0.8"
 end

data/lib/active_error.rb

@@ -5,8 +5,8 @@ require "active_error/exception_mock/template_error"
 require "active_error/exception_mock"
 require "active_error/captor"
 require "active_error/renderer"
-require "active_error/middleware"
 require "active_error/engine"
+require "active_error/railtie"
 require "active_error/version"
 
 module ActiveError

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: activeerror
 version: !ruby/object:Gem::Version
-  version: 1.0.7
+  version: 1.0.8
 platform: ruby
 authors:
 - Nick Pezza
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-01-10 00:00:00.000000000 Z
+date: 2024-01-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -66,7 +66,7 @@ files:
 - lib/active_error/exception_mock.rb
 - lib/active_error/exception_mock/default.rb
 - lib/active_error/exception_mock/template_error.rb
-- lib/active_error/middleware.rb
+- lib/active_error/railtie.rb
 - lib/active_error/renderer.rb
 - lib/active_error/version.rb
 - lib/activeerror.rb

data/lib/active_error/middleware.rb

@@ -1,17 +0,0 @@
-# frozen_string_literal: true
-
-module ActiveError
-  class Middleware
-    def initialize(app)
-      @app = app
-    end
-
-    def call(env)
-      @app.call(env)
-    rescue Exception => exception # rubocop:disable Lint/RescueException, Naming/RescuedExceptionsVariableName
-      Captor.new(exception:, request: ActionDispatch::Request.new(env)).capture
-
-      raise
-    end
-  end
-end