dislogger 0.1.8 → 0.1.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0b7ddd219546d4b47916d9e0cfd1303679ed1a3f4befa6675a7be486ada7de25
-  data.tar.gz: '001685aeefc4f82fd7a07f8a539ea9c737de5a1760eebdc0799660ea62c4087d'
+  metadata.gz: 9a274a53633279d495fd56b23a36c52730b7ca8e8c848f918e88be497b3d1dcf
+  data.tar.gz: 7fb1ce90ab63bbff72470682564c11772d5985e32195c46681daa8daf95c83ac
 SHA512:
-  metadata.gz: 40495b36577cce4bb00ea8ab38b74019b4792e2ca6452c1edd735f4042557403ab382bb70b08f94e4fb952b7e5240c1828057a5068f76aad4c1da5ba6be1fc41
-  data.tar.gz: 2d90483f3e1db1969355be2565384cde9f38f7048e5dd4f7573203a47bb814f3ba45bf11554bf9e0dd71c4751e0d7fcece7d547664afa8f2959bc98fbdba0eb9
+  metadata.gz: 64dc81ec91e4c71e6e168aadd71ab5a7d6abd4a632a7f192a7373cf47a816a4365f232bcf337b834c29c12a1abba3c71fd660305e2e8a66bde392b4d24342f9a
+  data.tar.gz: 2f55c155f9158f0252599b34353546d65bb14b9ea447efb35ce9cc59ea9b0a58a05777775cdfc32a352065eb42d9026c169f3cf4445d01a4ca767f3be3b690da

data/lib/dislogger/error_handler.rb

@@ -7,16 +7,30 @@ module Dislogger
     extend ActiveSupport::Concern
 
     included do
-      rescue_from StandardError, with: :handle_internal_server_error
-      rescue_from ActiveRecord::RecordNotFound, with: :handle_not_found
-      rescue_from ActiveRecord::RecordInvalid, with: :handle_unprocessable_entity
-      rescue_from ActionController::ParameterMissing, with: :handle_unprocessable_entity
-      rescue_from Pundit::NotAuthorizedError, with: :handle_forbidden if defined?(Pundit)
-      rescue_from CanCan::AccessDenied, with: :handle_forbidden if defined?(CanCan)
+      if ancestors.include?(ActionController::API) || ancestors.include?(ActionController::Base)
+        rescue_from Exception, with: :handle_exception
+        rescue_from StandardError, with: :handle_internal_server_error
+        rescue_from ActiveRecord::RecordNotFound, with: :handle_not_found
+        rescue_from ActiveRecord::RecordInvalid, with: :handle_unprocessable_entity
+        rescue_from ActionController::ParameterMissing, with: :handle_unprocessable_entity
+        rescue_from Pundit::NotAuthorizedError, with: :handle_forbidden if defined?(Pundit)
+        rescue_from CanCan::AccessDenied, with: :handle_forbidden if defined?(CanCan)
+      end
     end
 
     private
 
+    def handle_exception(exception)
+      Rails.logger.error("Dislogger caught exception: #{exception.class.name} - #{exception.message}")
+      Rails.logger.error(exception.backtrace.join("\n")) if exception.backtrace
+
+      notify_and_render_error(
+        message: "#{exception.class.name}: #{exception.message}",
+        status: :internal_server_error,
+        backtrace: exception.backtrace
+      )
+    end
+
     def handle_not_found(exception)
       notify_and_render_error(
         message: exception.message.presence || 'Resource not found',
@@ -42,7 +56,7 @@ module Dislogger
 
     def handle_internal_server_error(exception)
       notify_and_render_error(
-        message: 'Internal Server Error',
+        message: exception.message || 'Internal Server Error',
         status: :internal_server_error,
         backtrace: exception.backtrace
       )
@@ -51,12 +65,15 @@ module Dislogger
     def notify_and_render_error(message:, status:, details: nil, backtrace: nil)
       status_code = Rack::Utils::SYMBOL_TO_STATUS_CODE[status]
 
-      Notifiers::DiscordNotifier.new.notify(
+      notifier = Notifiers::DiscordNotifier.new
+      notification_result = notifier.notify(
         message: message,
         status: status_code,
         backtrace: backtrace
       )
 
+      Rails.logger.info("Dislogger notification result: #{notification_result}") if defined?(Rails)
+
       render_error(message, status, details)
     end
 

data/lib/dislogger/notifiers/discord_notifier.rb

@@ -4,22 +4,21 @@ module Dislogger
   module Notifiers
     class DiscordNotifier < BaseNotifier
       def notify(message:, status:, backtrace: nil)
-        return unless enabled? && @config.discord_webhook_url.present?
+        return false unless enabled? && @config.discord_webhook_url.present?
 
+        log_info("Attempting to send Discord notification")
         formatted_message = format_message(message, status, backtrace)
         send_notification(formatted_message)
       rescue StandardError => e
-        if defined?(Rails) && Rails.logger
-          Rails.logger.error("Discord notification failed: #{e.message}")
-        else
-          warn("Discord notification failed: #{e.message}")
-        end
+        log_error("Discord notification failed: #{e.message}")
+        log_error(e.backtrace.join("\n")) if e.backtrace
         false
       end
 
       protected
 
       def format_message(message, status, backtrace)
+        log_info("Formatting Discord message")
         Formatters::DiscordFormatter.new(
           message: message,
           status: status,
@@ -29,6 +28,7 @@ module Dislogger
       end
 
       def send_notification(payload)
+        log_info("Sending notification to Discord")
         response = HTTParty.post(
           @config.discord_webhook_url,
           body: payload.to_json,
@@ -37,19 +37,40 @@ module Dislogger
         
         unless response.success?
           error_message = "Discord API Error: #{response.code} - #{response.body}"
-          if defined?(Rails) && Rails.logger
-            Rails.logger.error(error_message)
-          else
-            warn(error_message)
-          end
+          log_error(error_message)
           return false
         end
         
+        log_info("Discord notification sent successfully")
         true
+      rescue StandardError => e
+        log_error("Failed to send Discord notification: #{e.message}")
+        log_error(e.backtrace.join("\n")) if e.backtrace
+        false
       end
 
       def enabled?
-        @config.enabled?
+        result = @config.enabled?
+        log_info("Dislogger enabled? #{result} (environment: #{@config.environment}, enabled_environments: #{@config.enabled_environments})")
+        result
+      end
+
+      private
+
+      def log_info(message)
+        if defined?(Rails) && Rails.logger
+          Rails.logger.info("[Dislogger] #{message}")
+        else
+          puts "[Dislogger] #{message}"
+        end
+      end
+
+      def log_error(message)
+        if defined?(Rails) && Rails.logger
+          Rails.logger.error("[Dislogger] #{message}")
+        else
+          warn "[Dislogger] #{message}"
+        end
       end
     end
   end

data/lib/dislogger/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Dislogger
-  VERSION = "0.1.8"
+  VERSION = "0.1.9"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: dislogger
 version: !ruby/object:Gem::Version
-  version: 0.1.8
+  version: 0.1.9
 platform: ruby
 authors:
 - Nelson