rails_exception_log 1.0.0 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f74b304353846663f7ce71910b4551a2ff998add94dcbfc1ec367cf8fd6a0cc1
-  data.tar.gz: f8f6dc75de6d73b215d47aa4dad59c07d0f91a2613b54c7f3d3bf050633b10fb
+  metadata.gz: 1837d104081234ee91cf50d520158a47f3ada29bde03c70eab21b9fd87a3495b
+  data.tar.gz: bc817c81266b7955a3026df62cc88f56391c58d9ab087cc5c8dc71a56a473df8
 SHA512:
-  metadata.gz: ac73adab5b4b0f3bd6000c0054e468081bd305f9543f36e00d2047e1d051af1312c1f4dbc7d61b33c47d8a2da3a32ab0e7d894ec8bf23d182d495e8779b1bdfd
-  data.tar.gz: 6ce8da2534567b8a8308af48eb6fe4db59777a02f702417fafba96f6ac61c85ff565ded94e0e141b3f26abdb0c6133aca5931afe5143bb2fd40e93ea0066b12b
+  metadata.gz: bb0f43621afbdc225e1db51822573e33c8bee27260cfc5f9546cfa0bc710c899bb052c8e0e47bd55ff93578d099485ce042478ab257d22e6e9dad17f880361cc
+  data.tar.gz: c11ac0b66730fa8741af48f007b197b604a3baae5535d5afd5363bc852d3c8af7af56c86320e69bb2844cadd1d5b8376808ca1c61de4761b8380ac537e3e6952

data/app/controllers/rails_exception_log/logged_exceptions_controller.rb

@@ -1,17 +1,22 @@
-require 'csv'
-
 module RailsExceptionLog
   class LoggedExceptionsController < ApplicationController
     before_action :load_exception, only: %i[show destroy resolve reopen ignore add_comment]
 
     def index
+      page = params[:page].to_i.positive? ? params[:page].to_i : 1
+      per_page = 25
+
       @exceptions = filtered_exceptions
                     .select(:id, :exception_class, :message, :controller_name,
                             :action_name, :request_method, :request_path,
                             :environment, :created_at, :status, :occurrence_count,
                             :last_occurred_at, :resolved_at)
                     .order(last_occurred_at: :desc)
-                    .page(params[:page])
+                    .limit(per_page)
+                    .offset((page - 1) * per_page)
+
+      total_count = filtered_exceptions.count
+      @total_pages = (total_count.to_f / per_page).ceil
 
       @exception_classes = RailsExceptionLog::LoggedException
                            .select(:exception_class)
@@ -54,31 +59,31 @@ module RailsExceptionLog
 
     def destroy
       @exception.destroy
-      redirect_to railsexceptionlog_exceptions_path,
+      redirect_to '/exceptions',
                   notice: 'Exception deleted successfully'
     end
 
     def resolve
       @exception.mark_resolved!
-      redirect_to railsexceptionlog_exception_path(@exception),
+      redirect_to "/exceptions/#{@exception.id}",
                   notice: 'Exception marked as resolved'
     end
 
     def reopen
       @exception.mark_open!
-      redirect_to railsexceptionlog_exception_path(@exception),
+      redirect_to "/exceptions/#{@exception.id}",
                   notice: 'Exception reopened'
     end
 
     def ignore
       @exception.update!(status: :ignored)
-      redirect_to railsexceptionlog_exception_path(@exception),
+      redirect_to "/exceptions/#{@exception.id}",
                   notice: 'Exception ignored'
     end
 
     def add_comment
       @exception.add_comment!(params[:comment], author: current_user_email)
-      redirect_to railsexceptionlog_exception_path(@exception),
+      redirect_to "/exceptions/#{@exception.id}",
                   notice: 'Comment added'
     end
 
@@ -88,7 +93,7 @@ module RailsExceptionLog
       else
         RailsExceptionLog::LoggedException.delete_all
       end
-      redirect_to railsexceptionlog_exceptions_path,
+      redirect_to '/exceptions',
                   notice: 'Exceptions cleared'
     end
 
@@ -129,6 +134,8 @@ module RailsExceptionLog
     end
 
     def export_to_csv(exceptions)
+      require 'csv'
+
       CSV.generate(headers: true) do |csv|
         csv << ['ID', 'Exception Class', 'Message', 'Controller', 'Action', 'Path', 'Method', 'Environment', 'Status',
                 'Occurrences', 'Created At']
@@ -151,7 +158,6 @@ module RailsExceptionLog
     end
 
     def current_user_email
-      # Override this method to return the current user's email
       'anonymous'
     end
   end

data/app/helpers/rails_exception_log/application_helper.rb

@@ -48,5 +48,11 @@ module RailsExceptionLog
       else 'bg-slate-100 text-slate-800'
       end
     end
+
+    def truncate_text(text, length: 50)
+      return '-' unless text
+
+      text.length > length ? "#{text[0...length]}..." : text
+    end
   end
 end

data/lib/rails_exception_log/engine.rb

@@ -4,17 +4,16 @@ module RailsExceptionLog
   class Engine < ::Rails::Engine
     isolate_namespace RailsExceptionLog
 
+    config.eager_load_paths << root.join('app/helpers')
+
     initializer 'rails_exception_log.assets' do |app|
       app.config.assets.paths << root.join('app/assets/stylesheets') if defined?(Sprockets) && app.config.assets
-
-      if defined?(Propshaft)
-        app.config.assets.loader = 'bun'
-        app.config.assets.build_paths << root.join('app/assets/builds')
-      end
     end
 
     initializer 'rails_exception_log.helpers' do
       ActionView::Base.include RailsExceptionLog::ApplicationHelper
+    rescue NameError, LoadError
+      # Helper may not be available
     end
 
     config.after_initialize do
@@ -24,3 +23,17 @@ module RailsExceptionLog
     end
   end
 end
+
+RailsExceptionLog::Engine.routes.draw do
+  get '/exceptions', to: 'logged_exceptions#index', as: :railsexceptionlog_exceptions
+  get '/exceptions/:id', to: 'logged_exceptions#show', as: :railsexceptionlog_exception
+  delete '/exceptions/:id', to: 'logged_exceptions#destroy'
+  delete '/exceptions', to: 'logged_exceptions#destroy_all'
+  get '/exceptions/export', to: 'logged_exceptions#export'
+
+  post '/exceptions/:id/resolve', to: 'logged_exceptions#resolve', as: :resolve_railsexceptionlog_exception
+  post '/exceptions/:id/reopen', to: 'logged_exceptions#reopen', as: :reopen_railsexceptionlog_exception
+  post '/exceptions/:id/ignore', to: 'logged_exceptions#ignore', as: :ignore_railsexceptionlog_exception
+  post '/exceptions/:id/add_comment', to: 'logged_exceptions#add_comment',
+                                      as: :add_comment_railsexceptionlog_exception
+end

data/lib/rails_exception_log/exception_loggable.rb

@@ -4,86 +4,39 @@ module RailsExceptionLog
   module ExceptionLoggable
     extend ActiveSupport::Concern
 
-    included do
-      after_action :log_exception_to_db, if: :exception_loggable?
-    end
-
     def log_exception_handler
-      log_exception_to_db
-      raise exception
-    end
+      Rails.logger.info '=' * 50
+      Rails.logger.info 'RAILS_EXCEPTION_LOG: Handler triggered!'
+      Rails.logger.info "Exception: #{exception.inspect}"
+      Rails.logger.info '=' * 50
 
-    private
-
-    def exception_loggable?
-      return false if exception.nil?
-      return false if is_a?(RailsExceptionLog::LoggedExceptionsController)
-      return false if ENV['RAILS_EXCEPTION_LOG_DISABLED'] == 'true'
-      return false unless rate_limit_allow?
-
-      true
-    end
-
-    def rate_limit_allow?
-      return true if Rails.env.development?
-
-      key = "exception_log_rate_limit:#{request&.remote_ip}"
-      count = Rails.cache.fetch(key, expires_in: 1.minute) do
-        0
-      end
-
-      max = RailsExceptionLog.max_requests_per_minute || 60
-      return false if count >= max
-
-      Rails.cache.write(key, count + 1, expires_in: 1.minute)
-      true
-    end
-
-    def log_exception_to_db
-      return unless exception || defined?(@exception) && @exception
-
-      exc = exception || @exception
-      return unless exc
+      exc = exception
+      return raise exc unless exc.present?
+      return raise exc if is_a?(RailsExceptionLog::LoggedExceptionsController)
+      return raise exc if ENV['RAILS_EXCEPTION_LOG_DISABLED'] == 'true'
 
       begin
-        user = current_user if respond_to?(:current_user)
-
-        logged_exception = RailsExceptionLog::LoggedException.create_or_update_with_fingerprint!(
-          exc,
-          request,
-          user: user
+        current_user if respond_to?(:current_user)
+
+        logged_exception = RailsExceptionLog::LoggedException.create!(
+          exception_class: exc.class.name,
+          message: exc.message,
+          backtrace: exc.backtrace&.join("\n"),
+          controller_name: controller_name,
+          action_name: action_name,
+          request_method: request&.method&.to_s,
+          request_path: request&.path,
+          request_params: request&.parameters&.to_h,
+          environment: Rails.env,
+          last_occurred_at: Time.current
         )
 
-        if respond_to?(:exception_data)
-          data = exception_data.is_a?(Proc) ? instance_exec(self, &exception_data) : send(exception_data)
-          logged_exception.update!(exception_data: data) if data.present?
-        end
-
-        RailsExceptionLog.after_log_exception.call(logged_exception) if RailsExceptionLog.after_log_exception
+        Rails.logger.info "RAILS_EXCEPTION_LOG: Created #{logged_exception.id}"
       rescue StandardError => e
-        Rails.logger.error "Failed to log exception: #{e.message}"
+        Rails.logger.error "RAILS_EXCEPTION_LOG Error: #{e.message}"
       end
-    end
-
-    def request_params_without_passwords
-      return {} unless request&.params
 
-      params_hash = request.params.deep_dup
-      filter_passwords(params_hash)
-      params_hash
-    end
-
-    def filter_passwords(hash)
-      hash.each do |key, value|
-        if key.to_s =~ /password|secret|token/i
-          hash[key] = '[FILTERED]'
-        elsif value.is_a?(Hash)
-          filter_passwords(value)
-        elsif value.is_a?(Array)
-          value.each { |v| filter_passwords(v) if v.is_a?(Hash) }
-        end
-      end
-      hash
+      raise exc
     end
   end
 end

data/lib/rails_exception_log/version.rb

@@ -1,3 +1,3 @@
 module RailsExceptionLog
-  VERSION = '1.0.0'
+  VERSION = '1.0.1'
 end

data/lib/rails_exception_log.rb

@@ -1,5 +1,6 @@
 require 'rails_exception_log/engine'
 require 'rails_exception_log/version'
+require 'rails_exception_log/exception_loggable'
 
 module RailsExceptionLog
   mattr_accessor :application_name, default: 'Rails Exception Log'
@@ -21,3 +22,7 @@ module RailsExceptionLog
     yield self
   end
 end
+
+ActiveSupport.on_load(:action_controller_base) do
+  include RailsExceptionLog::ExceptionLoggable
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rails_exception_log
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
 platform: ruby
 authors:
 - "Tamiru Hailu\n  "
@@ -91,7 +91,6 @@ files:
 - app/models/rails_exception_log/logged_exception.rb
 - app/views/rails_exception_log/logged_exceptions/index.html.erb
 - app/views/rails_exception_log/logged_exceptions/show.html.erb
-- config/routes.rb
 - db/migrate/20240101000000_create_rails_exception_log_logged_exceptions.rb
 - lib/generators/rails_exception_log/install_generator.rb
 - lib/rails_exception_log.rb

data/config/routes.rb

@@ -1,17 +0,0 @@
-module RailsExceptionLog
-  module Routes
-    def draw
-      get '/exceptions', to: 'logged_exceptions#index', as: :railsexceptionlog_exceptions
-      get '/exceptions/:id', to: 'logged_exceptions#show', as: :railsexceptionlog_exception
-      delete '/exceptions/:id', to: 'logged_exceptions#destroy'
-      delete '/exceptions', to: 'logged_exceptions#destroy_all'
-      get '/exceptions/export', to: 'logged_exceptions#export'
-
-      post '/exceptions/:id/resolve', to: 'logged_exceptions#resolve', as: :resolve_railsexceptionlog_exception
-      post '/exceptions/:id/reopen', to: 'logged_exceptions#reopen', as: :reopen_railsexceptionlog_exception
-      post '/exceptions/:id/ignore', to: 'logged_exceptions#ignore', as: :ignore_railsexceptionlog_exception
-      post '/exceptions/:id/add_comment', to: 'logged_exceptions#add_comment',
-                                          as: :add_comment_railsexceptionlog_exception
-    end
-  end
-end