activeerror 1.1.1 → 2.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cf8f5a1ee01022330a7ab7fd51b9d49e5bd39a1fa3ac4430c90e1321b7d4d2a1
-  data.tar.gz: f36c4e1b8052c14316e14075c9de5624ac3a09ee0816164f3d5e064f0bf6f55b
+  metadata.gz: f62d898e9e3ecefbee847720c5b0c494a175946ea248f380762ff9ff413a6e8e
+  data.tar.gz: ad82c8c07a1c2815c7a0ad58628a476f7e2d754682d8a09c45ce6e62cd930947
 SHA512:
-  metadata.gz: 81a4a5f81fd404d6d154ad6a65d27dc2f5690113d0b2f4b284d1a878295734526d9be88ae2cf0fd71c8c7d46b358b4e9aaf382abcb9578f16990ea2a6d46fe0d
-  data.tar.gz: ee9c6c4bca50c0dd8220b12c112bdfcc20d88b646cbc1c15cca1425d46c5a9768d9f1c9fc963c9febf8352f90936f0e16b4c868fbd6a565979b96c3c2c66f685
+  metadata.gz: fbbc8303ef12d3fc39d88d8320fb3b181eb08bd40cf8ebdd0da969bdd1f0f3e30f49d449838593656b319bbe07ed4b7630cec2adbadf58f36ed64043a078383f
+  data.tar.gz: 115594a0e228fc55d627c85ef2a0a081d6855e194f15a0741fbc5a54add7b2bd43cc9761ed5b6ed658243bef513d9eb8b3ce17bf7c9f18ff743dc7af5d1032eb

data/LICENSE.md

@@ -1,6 +1,6 @@
 The MIT License (MIT)
 
-Copyright (c) 2019 Nick Pezza
+Copyright (c) 2020 Nick Pezza
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

data/README.md

@@ -1,12 +1,14 @@
 # ActiveError
 
 One of the fundemental tools needed to take your Rails app to production is a
-way to track errors that are triggered. Unfortunately, theres no free, easy,
-open source way to track them for small or medium apps. Honeybadger, Sentry,
+way to track errors that are triggered. Unfortunately, there aren't many free,
+easy, open source ways to track them for small or medium apps. Honeybadger, Sentry,
 and AppSignal are great, but they are are closed source. With Sentry
 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.
+With Rails 8's ethos of No PAAS, there should be a way for new apps to start out
+with a basic error reporter and not be forced to pay a third party for one.
 
 ActiveError hooks into the [error reporting
 api](https://guides.rubyonrails.org/error_reporting.html) baked directly into
@@ -16,8 +18,8 @@ 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
 database (we attempt to group the same error together as Instances to reduce
 noise) and then we recreate the error and display it using the built in debug
-view from Rails. Once you've resolved the error you can click "Resolve" which
-will destroy the record.
+view from Rails (with a few font and styling tweaks). Once you've resolved the
+error you can click "Resolve" which will destroy the record.
 
 ![screenshot 1](https://github.com/npezza93/activeerror/blob/main/.github/screenshot1.png)
 ![screenshot 2](https://github.com/npezza93/activeerror/blob/main/.github/screenshot2.png)
@@ -39,22 +41,21 @@ Or install it yourself as:
 $ gem install activeerror
 ```
 
-And then install migrations:
+Run the installer and migrate:
 ```bash
-bin/rails generate active_error:install
-bin/rails rails db:migrate
+bin/rails active_error:install
+bin/rails db:migrate
 ```
 
-This also mounts a route in your routes file to view the errors at `/errors`.
-
+This will mount a route in your routes file to view the errors at `/errors`.
 
 ##### Config
 
-You can supply the string version of an error class to `ignored` to ignore
-classes of errors.
+You can supply a list of the string version of an error class to `ignored` to
+ignore classes of errors.
 
 ```ruby
-ActiveError.ignored = ["NoMethodError"]
+config.active_error.ignored = ["NoMethodError"]
 ```
 
 ## Development

data/Rakefile

@@ -1,20 +1,6 @@
 # frozen_string_literal: true
 
-begin
-  require "bundler/setup"
-rescue LoadError
-  puts "You must `gem install bundler` and `bundle install` to run rake tasks"
-end
-
-require "rdoc/task"
-
-RDoc::Task.new(:rdoc) do |rdoc|
-  rdoc.rdoc_dir = "rdoc"
-  rdoc.title    = "ActiveError"
-  rdoc.options << "--line-numbers"
-  rdoc.rdoc_files.include("README.md")
-  rdoc.rdoc_files.include("lib/**/*.rb")
-end
+require "bundler/setup"
 
 APP_RAKEFILE = File.expand_path("test/dummy/Rakefile", __dir__)
 load "rails/tasks/engine.rake"
@@ -23,12 +9,8 @@ load "rails/tasks/statistics.rake"
 
 require "bundler/gem_tasks"
 
-require "rake/testtask"
-
-Rake::TestTask.new(:test) do |t|
-  t.libs << "test"
-  t.pattern = "test/**/*_test.rb"
-  t.verbose = false
+task test: :environment do
+  sh("bin/rails test")
 end
 
 task default: :test

data/app/assets/stylesheets/activeerror/application.css

@@ -0,0 +1,175 @@
+@font-face {
+  font-family: 'Calibre';
+  font-style: normal;
+  font-weight: 400;
+  src: url('./calibre-regular.woff2') format('woff2');
+  font-display: swap;
+}
+
+@font-face {
+  font-family: 'Calibre';
+  font-style: italic;
+  font-weight: 400;
+  src: url('./calibre-regular-italic.woff2') format('woff2');
+  font-display: swap;
+}
+
+@font-face {
+  font-family: 'Calibre';
+  font-style: normal;
+  font-weight: 600;
+  src: url('./calibre-semibold.woff2') format('woff2');
+  font-display: swap;
+}
+
+@font-face {
+  font-family: 'Calibre';
+  font-style: italic;
+  font-weight: 600;
+  src: url('./calibre-semibold-italic.woff2') format('woff2');
+  font-display: swap;
+}
+
+@font-face {
+  font-family: 'Calibre';
+  font-style: normal;
+  font-weight: 700;
+  src: url('./calibre-bold.woff2') format('woff2');
+  font-display: swap;
+}
+
+.active_error {
+  body {
+    background-color: #fff;
+    color: #333;
+    margin: 0px;
+  }
+
+  body, p, ol, ul, td, input {
+    font-family: helvetica, verdana, arial, sans-serif;
+    font-size:   15px;
+    line-height: 18px;
+    font-family: 'Calibre';
+  }
+
+  header {
+    color: #D30001;
+    background: rgb(238, 231, 233);
+    padding: 7.5px 22.5px;
+    display: flex;
+    flex-direction: row;
+    justify-content: space-between;
+    align-items: center;
+  }
+
+  h1 {
+    flex: 1;
+    overflow-wrap: break-word;
+    margin: 0.2em 0;
+    line-height: 1.1em;
+    font-size: 30px;
+  }
+
+  #notice {
+    font-family: 'Calibre';
+    color: #000;
+    margin: 0px;
+    font-size: 20px;
+  }
+
+  table {
+    margin: 0;
+    border-collapse: collapse;
+    word-wrap:break-word;
+    table-layout: auto;
+    width: 100%;
+  }
+
+  .tr {
+    border-bottom: 3px solid rgba(38,27,35,0.1);
+    display: grid;
+    grid-template-columns: 10px 70px 1fr 215px 150px 180px;
+    align-items: center;
+
+    .all-cols {
+      grid-column: span 6 / span 6;
+      margin-bottom: 10px;
+
+      &> div {
+        display: grid;
+        grid-template-columns: 1fr 215px 300px;
+        margin-left: 110px;
+        flex-direction: row;
+
+        &> div {
+          padding-right: 5px;
+        }
+        div {
+          margin: 3px 0px;
+
+          a {
+            word-break: break-all;
+          }
+        }
+      }
+    }
+
+    a {
+      color: #261B23;
+      &:hover {
+        color: #D30001;
+      }
+    }
+
+    .td {
+      font-family: 'Calibre';
+      font-size: 20px;
+
+      &.chevron {
+        padding: 0px;
+        padding-left: 3px;
+        cursor: pointer;
+        font-size: 14px;
+        &.rotate {
+          transform: rotate(90deg);
+        }
+      }
+
+      &.location {
+        display: flex;
+        flex-direction: column;
+
+        span {
+          font-size: 16px;
+          margin-top: 10px;
+        }
+      }
+    }
+  }
+
+  .tr:nth-child(even) {
+    background: rgb(238, 231, 233);
+  }
+
+  .tr .td {
+    padding: 10px 30px;
+  }
+
+  .button-as-link {
+    border: none;
+    text-decoration: none;
+    cursor: pointer;
+    background: none;
+    font-size: 20px;
+    margin: 0;
+    padding: 7px 25px;
+    line-height: 18px;
+    color: #fff;
+    border-radius: 9999px;
+    background-color: #D30001;
+  }
+}
+
+.hidden {
+  display: none;
+}

data/app/controllers/active_error/faults_controller.rb

@@ -5,10 +5,14 @@ module ActiveError
     before_action :set_fault, only: %i(show destroy)
 
     def index
-      @faults = Fault.top_level
+      @faults = Fault.top_level.includes(:instances)
     end
 
     def show
+      instance = @fault.instances.last ||
+        @fault.instances.new(headers: {}, parameters: {})
+
+      render html: Renderer.new(instance:).body
     end
 
     def destroy

data/app/models/active_error/fault.rb

@@ -2,12 +2,14 @@
 
 module ActiveError
   class Fault < ApplicationRecord
+    self.filter_attributes = [:backtrace]
+
     has_one :parent_cause, class_name: "ActiveError::Fault",
                            foreign_key: :cause_id, inverse_of: :cause,
                            dependent: :restrict_with_error
     belongs_to :cause, optional: true, class_name: "ActiveError::Fault",
                        dependent: :destroy
-    has_many :instances, dependent: :destroy
+    has_many :instances, dependent: :delete_all
 
     store :options, accessors: [:template], coder: YAML,
                     yaml: { unsafe_load: true }
@@ -39,12 +41,16 @@ module ActiveError
       super(Marshal.dump(new_template))
     end
 
+    def location
+      Rails.backtrace_cleaner.clean(backtrace).first
+    end
+
     private
 
     def controller_display
       return unless controller?
 
-      " in #{controller.camelize}Controller"
+      " in #{controller.camelize}"
     end
 
     def action_display

data/app/views/active_error/faults/index.html.erb

@@ -3,16 +3,39 @@
   <p id="notice"><%= notice %><p>
 </header>
 
-<table>
-  <tbody>
-    <% @faults.each do |fault| %>
-      <tr>
-        <td>#<%= fault.id %></td>
-        <td><%= link_to fault.title, fault %></td>
-        <td class="w-15"><%= time_ago_in_words(fault.created_at).capitalize %> ago</td>
-        <td class="w-15"><%= fault.occurrences %></td>
-        <td class="w-5"><%= button_to 'Resolve', fault, method: :delete, form: { onSubmit: "return confirm('Are you sure?')" }, class: "button-as-link" %></td>
-      </tr>
-    <% end %>
-  </tbody>
-</table>
+<div class="table">
+  <% @faults.each do |fault| %>
+    <div class="tr">
+      <div class="td chevron">▶</div>
+      <div class="td">#<%= fault.id %></div>
+      <div class="td location">
+        <%= link_to fault.title, fault_path(fault) %>
+
+        <% if fault.location.present? %>
+          <span><%= fault.location %></span>
+        <% end %>
+      </div>
+      <div class="td"><%= time_ago_in_words(fault.created_at).capitalize %> ago</div>
+      <div class="td"><%= fault.occurrences %></div>
+      <div class="td"><%= button_to 'Resolve', fault, method: :delete, form: { onSubmit: "return confirm('Are you sure?')" }, class: "button-as-link" %></div>
+    <div class="all-cols hidden">
+      <% fault.instances.each do |instance| %>
+        <div>
+          <div><%= link_to instance.url, fault_instance_path(fault, instance) %></div>
+          <div><%= time_ago_in_words(instance.created_at).capitalize %> ago</div>
+          <div><%= instance.user_agent_display %></div>
+        </div>
+      <% end %>
+    </div>
+    </div>
+  <% end %>
+</div>
+
+<script>
+  document.querySelectorAll(".chevron").forEach((el) => {
+    el.addEventListener("click", (event) => {
+      event.target.parentElement.querySelector(".all-cols").classList.toggle("hidden")
+      event.target.classList.toggle("rotate")
+    })
+  })
+</script>

data/app/views/layouts/active_error/application.html.erb

@@ -5,96 +5,7 @@
     <%= csrf_meta_tags %>
     <%= csp_meta_tag %>
 
-    <style>
-      .active_error body {
-        background-color: #FAFAFA;
-        color: #333;
-        color-scheme: light dark;
-        supported-color-schemes: light dark;
-        margin: 0px;
-      }
-
-      @media (prefers-color-scheme: dark) {
-        .active_error body {
-          background-color: #222;
-          color: #ECECEC;
-        }
-      }
-
-      .active_error body, .active_error p, .active_error ol, .active_error ul, .active_error td {
-        font-family: helvetica, verdana, arial, sans-serif;
-        font-size: 13px;
-        line-height: 18px;
-      }
-
-      .active_error .button-as-link {
-        border: none;
-        text-decoration: underline;
-        cursor: pointer;
-        background: none;
-        font-size: 15px;
-        margin: 0;
-        padding: 0;
-        line-height: 18px;
-      }
-
-      .active_error a, .active_error .button-as-link { color: #980905; }
-      .active_error a:visited, .active_error .button-as-link:visited { color: #666; }
-      .active_error a.trace-frames {
-        color: #666;
-        overflow-wrap: break-word;
-      }
-      .active_error a:hover, .active_error .button-as-link:hover, .active_error a.trace-frames.selected { color: #C00; }
-      .active_error a.summary:hover { color: #FFF; }
-
-      .active_error th {
-        padding-bottom: 5px;
-      }
-
-      .active_error td {
-        padding: 0 5px 7px;
-      }
-
-      .active_error #notice {
-        color: green;
-        margin: 0px;
-      }
-
-      .active_error header {
-        color: #f0f0f0;
-        background: #C00;
-        padding: 0.5em 1.5em;
-        display: flex;
-        flex-direction: row;
-        align-items: center;
-        margin-bottom: 0.83em;
-      }
-
-      .active_error h1 {
-        flex: 1;
-        overflow-wrap: break-word;
-        margin: 0.2em 0;
-        line-height: 1.1em;
-        font-size: 2em;
-      }
-
-      .active_error table {
-        padding: 0 1.5em;
-        width: 100%;
-      }
-
-      .active_error table tr td:first-of-type {
-        padding-left: 0;
-      }
-
-      .active_error .w-5 {
-        width: 5%;
-      }
-
-      .active_error .w-15 {
-        width: 15%;
-      }
-    </style>
+    <%= stylesheet_link_tag "activeerror/application" %>
   </head>
   <body><%= yield %></body>
 </html>

data/app/views/layouts/active_error/rescue.html.erb

@@ -0,0 +1,278 @@
+<!DOCTYPE html>
+<html lang="en">
+<head>
+  <meta charset="utf-8" />
+  <meta name="viewport" content="width=device-width, initial-scale=1">
+  <meta name="turbo-visit-control" content="reload">
+  <title>Action Controller: Exception caught</title>
+  <%= stylesheet_link_tag "activeerror/application" %>
+  <link rel="preconnect" href="https://fonts.googleapis.com">
+  <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin>
+  <link href="https://fonts.googleapis.com/css2?family=Fira+Code:wght@400&display=swap" rel="stylesheet">
+  <style>
+    body {
+      background-color: #FAFAFA;
+      color: #333;
+      color-scheme: light dark;
+      supported-color-schemes: light dark;
+      margin: 0px;
+    }
+
+    pre, code {
+      font-family: "JetBrains Mono", monospace;
+      font-optical-sizing: auto;
+      font-weight: 400;
+      font-style: normal;
+    }
+
+    body, p, ol, ul, td {
+      font-family: "Calibre", helvetica, verdana, arial, sans-serif;
+      font-size:   15px;
+      line-height: 18px;
+    }
+
+    pre {
+      font-size: 11px;
+      white-space: pre-wrap;
+    }
+
+    pre.box {
+      border: 1px solid #EEE;
+      padding: 10px;
+      margin: 0px;
+      width: 958px;
+    }
+
+    header {
+      color: #F0F0F0;
+      background: #C00;
+      padding: 0.5em 1.5em;
+    }
+
+    h1 {
+      overflow-wrap: break-word;
+      margin: 0.2em 0;
+      line-height: 1.1em;
+      font-size: 2em;
+    }
+
+  header {
+    color: #D30001;
+    background: rgb(238, 231, 233);
+    padding: 7.5px 22.5px;
+    display: flex;
+    flex-direction: row;
+    justify-content: space-between;
+    align-items: center;
+  }
+
+  h1 {
+    flex: 1;
+    overflow-wrap: break-word;
+    margin: 0.2em 0;
+    line-height: 1.1em;
+    font-size: 30px;
+    font-family: "Calibre";
+  }
+
+    h2 {
+      color: #C00;
+      line-height: 25px;
+    }
+
+    code.traces {
+      font-size: 11px;
+    }
+
+    .response-heading, .request-heading {
+      margin-top: 30px;
+      font-family: "Calibre";
+      font-size: 22px;
+    }
+
+    .exception-message {
+      padding: 8px 0;
+    }
+
+    .exception-message .message {
+      margin-bottom: 8px;
+      line-height: 25px;
+      font-size: 24px;
+      font-weight: 600;
+      color: #C00;
+      font-family: "Calibre";
+    }
+
+    .details {
+      border: 1px solid #D0D0D0;
+      border-radius: 4px;
+      margin: 1em 0px;
+      display: block;
+      max-width: 978px;
+    }
+
+    .summary {
+      padding: 8px 15px;
+      border-bottom: 1px solid #D0D0D0;
+      display: block;
+    }
+
+    a.summary {
+      color: #F0F0F0;
+      text-decoration: none;
+      background: #C52F24;
+      border-bottom: none;
+    }
+
+    .details pre {
+      margin: 5px;
+      border: none;
+    }
+
+    #container {
+      box-sizing: border-box;
+      width: 100%;
+      padding: 0 1.5em;
+    }
+
+    .source * {
+      margin: 0px;
+      padding: 0px;
+    }
+
+    .source {
+      border: 1px solid #D9D9D9;
+      background: #ECECEC;
+      max-width: 978px;
+    }
+
+    .source pre {
+      padding: 10px 0px;
+      border: none;
+    }
+
+    .source .data {
+      font-size: 80%;
+      overflow: auto;
+      background-color: #FFF;
+    }
+
+    .info {
+      padding: 0.5em;
+    }
+
+    .source .data .line_numbers {
+      background-color: #ECECEC;
+      color: #555;
+      padding: 1em .5em;
+      border-right: 1px solid #DDD;
+      text-align: right;
+    }
+
+    .line {
+      padding-left: 10px;
+      white-space: pre;
+    }
+
+    .line:hover {
+      background-color: #F6F6F6;
+    }
+
+    .line.active {
+      background-color: #FCC;
+    }
+
+    .error_highlight {
+      display: inline-block;
+      background-color: #FF9;
+      text-decoration: #F00 wavy underline;
+    }
+
+    .error_highlight_tip {
+      color: #666;
+      padding: 2px 2px;
+      font-size: 10px;
+    }
+
+    .button_to {
+      display: inline-block;
+      margin-top: 0.75em;
+      margin-bottom: 0.75em;
+    }
+
+    .hidden {
+      display: none;
+    }
+
+    .correction {
+      list-style-type: none;
+    }
+
+    input[type="submit"] {
+      color: white;
+      background-color: #C00;
+      border: none;
+      border-radius: 12px;
+      box-shadow: 0 3px #F99;
+      font-size: 13px;
+      font-weight: bold;
+      margin: 0;
+      padding: 10px 18px;
+      cursor: pointer;
+      -webkit-appearance: none;
+    }
+    input[type="submit"]:focus,
+    input[type="submit"]:hover {
+      opacity: 0.8;
+    }
+    input[type="submit"]:active {
+      box-shadow: 0 2px #F99;
+      transform: translateY(1px)
+    }
+
+    a { color: #980905; }
+    a:visited { color: #666; }
+    a.trace-frames {
+      color: #666;
+      overflow-wrap: break-word;
+    }
+    a:hover, a.trace-frames.selected { color: #C00; }
+    a.summary:hover { color: #FFF; }
+
+    h2.response-heading {
+      display: none;
+      &+p {
+        display: none;
+        &+pre {
+          display: none;
+        }
+      }
+    }
+    <%= yield :style %>
+  </style>
+
+  <script>
+    var toggle = function(id) {
+      document.getElementById(id).classList.toggle('hidden');
+      return false;
+    }
+    var show = function(id) {
+      document.getElementById(id).style.display = 'block';
+    }
+    var hide = function(id) {
+      document.getElementById(id).style.display = 'none';
+    }
+    var toggleSessionDump = function() {
+      return toggle('session_dump');
+    }
+    var toggleEnvDump = function() {
+      return toggle('env_dump');
+    }
+  </script>
+</head>
+<body>
+
+  <%= yield %>
+
+</body>
+</html>

data/config/routes.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 ActiveError::Engine.routes.draw do
-  resources :faults, only: %i(index show destroy) do
+  resources :faults, only: %i(show index destroy) do
     scope module: :faults do
       resources :instances, only: :show
     end

data/db/migrate/20200727220359_create_active_error_faults.rb

@@ -4,14 +4,14 @@ class CreateActiveErrorFaults < ActiveRecord::Migration[7.1]
   def change # rubocop:disable Metrics/AbcSize
     create_table :active_error_faults do |t|
       t.belongs_to :cause
-      t.binary :backtrace, lmit: 512.megabytes
-      t.binary :backtrace_locations, lmit: 512.megabytes
+      t.binary :backtrace, limit: 512.megabytes
+      t.binary :backtrace_locations, limit: 512.megabytes
       t.string :klass
       t.text :message
       t.string :controller
       t.string :action
       t.integer :instances_count
-      t.text :blamed_files, lmit: 512.megabytes
+      t.text :blamed_files, limit: 512.megabytes
       t.text :options
       t.index %i(klass backtrace message)
 

data/lib/active_error/captor.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module ActiveError
-  BacktraceLocation = Struct.new(:absolute_path, :base_label, :to_s, :label, # rubocop:disable Lint/StructNewOverride
+  BacktraceLocation = Struct.new(:absolute_path, :base_label, :to_s, :label,
                                  :lineno, :path) do
     def inspect = to_s
 
@@ -42,7 +42,8 @@ module ActiveError
     end
 
     def template?
-      exception.class.is_a?(ActionView::Template::Error)
+      exception.is_a?(ActionView::Template::Error) ||
+        exception.class.is_a?(ActionView::Template::Error)
     end
 
     def template

data/lib/active_error/engine.rb

@@ -7,9 +7,19 @@ module ActiveError
   class Engine < ::Rails::Engine
     isolate_namespace ActiveError
 
-    initializer "active_error.middleware" do |app|
-      app.config.middleware.use ActiveError::Middleware
+    config.active_error = ActiveSupport::OrderedOptions.new
 
+    initializer "active_error.error_reporter" do |_app|
+      Rails.error.subscribe(::ActiveError::ErrorSubscriber.new)
+    end
+
+    initializer "active_error.config" do
+      config.active_error.each do |name, value|
+        ActiveError.public_send(:"#{name}=", value)
+      end
+    end
+
+    initializer "active_error.context" do
       ActionController::Base.before_action do
         Rails.error.set_context(active_error_request: request)
       end

data/lib/active_error/error_subscriber.rb

@@ -0,0 +1,12 @@
+# frozen_string_literal: true
+
+module ActiveError
+  class ErrorSubscriber
+    def report(exception, context:, handled:, **_opts)
+      return if !ActiveError.enabled? || handled ||
+        ActiveError.ignored_classes.include?(exception.class.to_s)
+
+      Captor.new(exception:, request: context[:active_error_request]).capture
+    end
+  end
+end

data/lib/active_error/renderer.rb

@@ -8,7 +8,11 @@ module ActiveError
     end
 
     def body
-      @body ||= template.render(template: file, layout: "rescues/layout")
+      @body ||= begin
+        set_lookup_context
+
+        template.render(template: file, layout: "active_error/rescue")
+      end
     end
 
     private
@@ -16,27 +20,39 @@ module ActiveError
     attr_reader :exception, :instance
 
     delegate :trace_to_show, :source_to_show_id, :source_extracts,
-             :traces, to: :wrapper
-
-    def backtrace_cleaner
-      Rails.backtrace_cleaner
-    end
+             :traces, to: :exception_wrapper
 
-    def wrapper
-      @wrapper ||=
-        ActionDispatch::ExceptionWrapper.new(backtrace_cleaner, exception)
+    def exception_wrapper
+      @exception_wrapper ||=
+        ActionDispatch::ExceptionWrapper.new(Rails.backtrace_cleaner, exception)
     end
 
     def file
-      "rescues/#{wrapper.rescue_template}"
+      "rescues/#{exception_wrapper.rescue_template}"
     end
 
     def template
-      ActionDispatch::DebugView.new(
-        request: instance.request, exception_wrapper: wrapper,
-        exception: wrapper.exception, traces:,
-        show_source_idx: source_to_show_id, trace_to_show:, source_extracts:
-      )
+      @template ||=
+        ActionDispatch::DebugView.
+        new(request: instance.request, exception_wrapper:, traces:,
+            exception: exception_wrapper.exception, trace_to_show:,
+            source_extracts:, show_source_idx: source_to_show_id,)
+    end
+
+    def set_lookup_context
+      template.define_singleton_method :lookup_context= do |new_context|
+        @lookup_context = new_context
+        @view_renderer = ActionView::Renderer.new @lookup_context
+      end
+
+      template.lookup_context = new_lookup_context
+    end
+
+    def new_lookup_context
+      ActionView::LookupContext.new([
+        *ActionDispatch::DebugView::RESCUES_TEMPLATE_PATHS.dup,
+        ActiveError::Engine.root.join("app/views/layouts").to_s
+      ])
     end
   end
 end

data/lib/active_error/version.rb

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

data/lib/active_error.rb

@@ -1,15 +1,45 @@
 # frozen_string_literal: true
 
+require "active_error/error_subscriber"
 require "active_error/exception_mock/default"
 require "active_error/exception_mock/template_error"
 require "active_error/exception_mock"
 require "active_error/captor"
-require "active_error/middleware"
 require "active_error/renderer"
 require "active_error/engine"
-require "active_error/railtie"
 require "active_error/version"
 
 module ActiveError
-  mattr_accessor :ignored
+  mattr_accessor :ignored, :enabled
+
+  IGNORE_DEFAULT = [
+    "AbstractController::ActionNotFound",
+    "ActionController::BadRequest",
+    "ActionController::InvalidAuthenticityToken",
+    "ActionController::InvalidCrossOriginRequest",
+    "ActionController::MethodNotAllowed",
+    "ActionController::NotImplemented",
+    "ActionController::ParameterMissing",
+    "ActionController::RoutingError",
+    "ActionController::UnknownAction",
+    "ActionController::UnknownFormat",
+    "ActionDispatch::Http::MimeNegotiation::InvalidType",
+    "ActionController::UnknownHttpMethod",
+    "ActionDispatch::Http::Parameters::ParseError",
+    "ActiveRecord::RecordNotFound"
+  ].freeze
+
+  class << self
+    def ignored_classes
+      ignored.to_a + IGNORE_DEFAULT
+    end
+
+    def enabled?
+      if enabled.nil?
+        !Rails.env.local?
+      else
+        enabled.present?
+      end
+    end
+  end
 end

data/lib/tasks/active_error_tasks.rake

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+desc "Copy over the migrations and mount route for ActiveError"
+namespace :active_error do
+  task install: :environment do
+    Rails::Command.invoke :generate, ["active_error:install"]
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: activeerror
 version: !ruby/object:Gem::Version
-  version: 1.1.1
+  version: 2.0.0
 platform: ruby
 authors:
 - Nick Pezza
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-02-09 00:00:00.000000000 Z
+date: 2024-10-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '7.1'
+        version: '7.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '7.1'
+        version: '7.2'
 - !ruby/object:Gem::Dependency
   name: useragent
   requirement: !ruby/object:Gem::Requirement
@@ -48,6 +48,12 @@ files:
 - LICENSE.md
 - README.md
 - Rakefile
+- app/assets/stylesheets/activeerror/application.css
+- app/assets/stylesheets/activeerror/calibre-bold.woff2
+- app/assets/stylesheets/activeerror/calibre-regular-italic.woff2
+- app/assets/stylesheets/activeerror/calibre-regular.woff2
+- app/assets/stylesheets/activeerror/calibre-semibold-italic.woff2
+- app/assets/stylesheets/activeerror/calibre-semibold.woff2
 - app/controllers/active_error/application_controller.rb
 - app/controllers/active_error/faults/instances_controller.rb
 - app/controllers/active_error/faults_controller.rb
@@ -55,23 +61,23 @@ files:
 - app/models/active_error/fault.rb
 - app/models/active_error/instance.rb
 - app/views/active_error/faults/index.html.erb
-- app/views/active_error/faults/show.html.erb
 - app/views/layouts/active_error/application.html.erb
+- app/views/layouts/active_error/rescue.html.erb
 - config/routes.rb
 - db/migrate/20200727220359_create_active_error_faults.rb
 - db/migrate/20200727225318_create_active_error_instances.rb
 - lib/active_error.rb
 - lib/active_error/captor.rb
 - lib/active_error/engine.rb
+- lib/active_error/error_subscriber.rb
 - 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
 - lib/generators/active_error/install/install_generator.rb
+- lib/tasks/active_error_tasks.rake
 homepage: https://github.com/npezza93/activeerror
 licenses:
 - MIT
@@ -85,14 +91,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 3.1.0
+      version: 3.2.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.3
+rubygems_version: 3.5.18
 signing_key:
 specification_version: 4
 summary: Rails exception logger

data/app/views/active_error/faults/show.html.erb

@@ -1,17 +0,0 @@
-<header>
-  <h1>Instances of <%= @fault.title %> (Fault #<%= @fault.id %>)</h1>
-  <p id="notice"><%= notice %><p>
-</header>
-
-<table>
-  <tbody>
-    <% @fault.instances.each do |instance| %>
-      <tr>
-        <td>#<%= instance.id %></td>
-        <td><%= link_to instance.url, fault_instance_path(@fault, instance) %></td>
-        <td class="w-15"><%= instance.user_agent_display %></td>
-        <td class="w-15"><%= time_ago_in_words(instance.created_at).capitalize %> ago</td>
-      </tr>
-    <% end %>
-  </tbody>
-</table>

data/lib/active_error/middleware.rb

@@ -1,19 +0,0 @@
-# frozen_string_literal: true
-
-module ActiveError
-  class Middleware
-    def initialize(app)
-      @app = app
-    end
-
-    def call(env)
-      @app.call(env)
-    rescue StandardError => e
-      unless Rails.application.config.reloading_enabled?
-        Rails.error.report(e, handled: false,
-                              source: "application.action_dispatch")
-      end
-      raise
-    end
-  end
-end

data/lib/active_error/railtie.rb

@@ -1,18 +0,0 @@
-# frozen_string_literal: true
-
-module ActiveError
-  class ErrorSubscriber
-    def report(exception, context:, handled:, **_opts)
-      return if Rails.env.development? || handled ||
-        ActiveError.ignored.to_a.include?(exception.class.to_s)
-
-      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