eras 0.2.4 → 0.2.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bcabf38f85c19c28d26ec76db463d508a1bcd1f2a8ee97c2ba3c880f8fe8ffd9
-  data.tar.gz: 96f5730869b8dc2651dedae5032407d06503ee92e8b4e2fa8e9d282685061436
+  metadata.gz: eac092a75031412c1ac1310c0aee54020655f0a5949eaab51b9757a576d81c2e
+  data.tar.gz: 32e1f8e528e35cecb3817c756bc3b36744ac1db9a0ef8b401f43f01655078614
 SHA512:
-  metadata.gz: 861833d3957291c5a0ac83bbce52af9ded69fc40448c01542b85e2b1cfee45bf84f08b6a2ee3f63c9de7d714d84be69a2c03e4c5131da99ad552f090b9505666
-  data.tar.gz: 3736ffd2be3718e0fcf380c2f5f6c01b94cac16af6186e1624693d57349fd09e2135e12e7d8c509cda79767435812bcf47756b67141914da5c249a9c74530906
+  metadata.gz: 55f51be8a2e27e43404b7f198fdb00bd7dd87d77cfac949dcc0b161fa9c222f0cb523e02f1041cb346c77f20eaecc3fc6638c74ede4f2eb99dfd0fcdf28d8eba
+  data.tar.gz: ebc26565ef50af9d8105c70ab08165ebf6fb8e92ae55d2fe58e05dc420bb70b63e7e2a72f463926d21fc70964d88ce69853b79668a9fd1f41d9f96d832f8b9e7

data/app/controllers/eras/errors_controller.rb

@@ -1,28 +1,28 @@
 module Eras
   class ErrorsController < ApplicationController
     before_action :set_file_attrs
+    before_action :set_environment, only: [:index, :show]
 
     def index
-      @environment = params[:environment] || 'development'
-      # TODO: We need to actually respect the adapter that's used,
-      # and not just assume it's the filesystem adapter.
-      @errors = Eras::Adapters::FileSystem.new.read_errors.filter { |e| e['context']['rails_env'] == @environment }
+      @errors = Eras.adapter.read_errors.filter { |e| e['context']['rails_env'] == @environment }
     end
 
     def show
-      @error = Eras::Adapters::FileSystem.new.read_error(params[:id])
+      @error = Eras.adapter.read_error(params[:id])
     end
 
     def destroy
-      Eras::Adapters::FileSystem.new.destroy_file
+      # TODO: rename to be adapter agnostic
+      Eras.adapter.destroy_file
+    end
+
+    def set_environment
+      @environment = params[:environment] || 'development'
     end
 
     def set_file_attrs
-      # TODO: We need to store the active adapter as a singleton somewhere
-      # so we can use it places instead of creating new ones.
-      adapter = Eras::Adapters::FileSystem.new
-      @file_path = adapter.path
-      @file_size = adapter.file_size
+      @file_path = Eras.adapter.path
+      @file_size = Eras.adapter.file_size
     end
   end
 end

data/app/views/eras/errors/index.html.erb

@@ -31,7 +31,7 @@
                                     <%= error.dig("error", "message") %></td>
                                 <td>
                                     <div class="flex space-x-2 justify-center">
-                                        <a href="<%= error_path(error['id'])%>"
+                                        <a href="<%= error_path(id: error['id'], environment: @environment)%>"
                                             class="inline-block px-6 py-2.5 bg-blue-600 text-white font-medium text-xs leading-tight uppercase rounded shadow-md hover:bg-blue-700 hover:shadow-lg focus:bg-blue-700 focus:shadow-lg focus:outline-none focus:ring-0 active:bg-blue-800 active:shadow-lg transition duration-150 ease-in-out">View</a>
                                     </div>
                                 </td>

data/app/views/eras/errors/show.html.erb

@@ -3,7 +3,7 @@
 
   <p><%= @error.dig('error', 'message') %></p>
     <a
-    href="<%= errors_path %>"
+    href="<%= errors_path(environment: @environment) %>"
     class="inline-block px-6 py-2.5 mt-4 bg-blue-600 text-white font-medium text-xs leading-tight uppercase rounded shadow-md hover:bg-blue-700 hover:shadow-lg focus:bg-blue-700 focus:shadow-lg focus:outline-none focus:ring-0 active:bg-blue-800 active:shadow-lg transition duration-150 ease-in-out"
     data-mdb-ripple="true"
     data-mdb-ripple-color="light"
@@ -12,6 +12,6 @@
   </a>
   <hr class="my-6 border-gray-300" />
   <pre>
-    <code class="whitespace-pre-line text-xs"><%= @error.dig('error', 'backtrace').strip %></code>
+    <code class="whitespace-pre-line text-xs"><%= @error.dig('error', 'backtrace')&.strip %></code>
   </pre>
 </div>

data/lib/eras/adapters/file_system.rb

@@ -1,16 +1,21 @@
 module Eras
   module Adapters
     class FileSystem
+      class Error < StandardError; end
+
       include ActionView::Helpers::NumberHelper
       attr_reader :path
 
-      def initialize(path = nil)
-        if path.nil?
+      def initialize(dir = nil)
+        if dir.nil?
           dir = Rails.root.join("tmp", "eras")
+
           Dir.mkdir(dir) unless File.exists?(dir)
         end
-        # TODO: Actually use the path
-        @path = Rails.root.join("tmp", "eras", "errors.json")
+
+        raise Error, "Directory #{dir} does not exist" unless File.exists?(dir)
+
+        @path = File.join(dir, "errors.json")
       end
 
       def write_error(data)

data/lib/eras/config.rb

@@ -0,0 +1,12 @@
+require "eras/adapters/file_system"
+
+module Eras
+  class Config
+    attr_accessor :adapter, :ignored_error_classes
+
+    def initialize
+      @adapter = Eras::Adapters::FileSystem.new
+      @ignored_error_classes = []
+    end
+  end
+end

data/lib/eras/error_reporter.rb

@@ -1,10 +1,8 @@
 module Eras
   class ErrorReporter
-    def initialize(adapter: Eras::Adapters::FileSystem.new)
-      @adapter = adapter
-    end
-
     def report(error, handled:, severity:, context:, source: nil)
+      return if Eras.config.ignored_error_classes.include?(error.class.name)
+
       context = context.merge(rails_env: ::Rails.env).transform_values(&:to_s)
 
       data = {
@@ -15,10 +13,10 @@ module Eras
         },
         handled: handled,
         severity: severity,
-        context: context, # ActionController.to_s causes stack overflow
+        context: context,
         source: source
       }
-      @adapter.write_error(data)
+      Eras.adapter.write_error(data)
     end
   end
 end

data/lib/eras/version.rb

@@ -1,3 +1,3 @@
 module Eras
-  VERSION = "0.2.4"
+  VERSION = "0.2.6"
 end

data/lib/eras.rb

@@ -1,7 +1,17 @@
 require "eras/version"
 require "eras/engine"
-require "eras/adapters/file_system"
+require "eras/config"
 
 module Eras
-  # Your code goes here...
+  def self.config
+    @config ||= Eras::Config.new
+  end
+
+  def self.configure
+    yield config
+  end
+
+  def self.adapter
+    config.adapter
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: eras
 version: !ruby/object:Gem::Version
-  version: 0.2.4
+  version: 0.2.6
 platform: ruby
 authors:
 - Bobby McDonald
@@ -65,6 +65,7 @@ files:
 - config/tailwind.config.js
 - lib/eras.rb
 - lib/eras/adapters/file_system.rb
+- lib/eras/config.rb
 - lib/eras/engine.rb
 - lib/eras/error_reporter.rb
 - lib/eras/version.rb