exception_handler 0.7.5 → 0.8.0.1

data/app/mailers/exception_handler/exception_mailer.rb

@@ -1,17 +1,17 @@
-module ExceptionHandler
-  class ExceptionMailer < ActionMailer::Base
-
-      # Layout
-      layout "mailers/layouts/mailer"
-
-      # Defaults
-      default from: 			    ExceptionHandler.config.email
-      default template_path: 	"exception_handler/mailers" # => http://stackoverflow.com/a/18579046/1143732
-
-      def new_exception e
-      	@exception = e
-        mail to: ExceptionHandler.config.email
-        Rails.logger.info "Exception Sent To → #{ExceptionHandler.config.email}"
-      end
-  end
-end
+module ExceptionHandler
+  class ExceptionMailer < ActionMailer::Base
+
+      # Layout
+      layout "mailer"
+
+      # Defaults
+      default from: 			    ExceptionHandler.config.email
+      default template_path: 	"exception_handler/mailers" # => http://stackoverflow.com/a/18579046/1143732
+
+      def new_exception e
+      	@exception = e
+        mail to: ExceptionHandler.config.email
+        Rails.logger.info "Exception Sent To → #{ExceptionHandler.config.email}"
+      end
+  end
+end

data/app/models/exception_handler/exception.rb

@@ -1,191 +1,200 @@
-module ExceptionHandler
-
-  ############################################################
-  ############################################################
-
-    # => Search Bots
-    # => Used in "Exception" class
-    BOTS = %w(Baidu Gigabot Googlebot libwww-per lwp-trivial msnbot SiteUptime Slurp Wordpress ZIBB ZyBorg Yandex Jyxobot Huaweisymantecspider ApptusBot)
-
-    # => Attributes
-    # => Determine schema etc
-    ATTRS = %i(class_name status message trace target referrer params user_agent)
-
-  ############################################################
-  ############################################################
-
-    # => Class (inheritance dependent on whether db option is available)
-    self::Exception = Class.new (ExceptionHandler.config.try(:db) ? ActiveRecord::Base : Object) do
-
-      # => Include individual elements
-      # => Only required if no db present (no ActiveRecord)
-      if ExceptionHandler.config.try(:db)
-
-        # => Set Attrs
-        def initialize attributes={}
-            super
-            ATTRS.each do |type|
-              self[type] = eval type.to_s
-            end
-        end
-
-      else
-
-        # => AciveModel
-        include ActiveModel::Model
-        include ActiveModel::Validations
-
-        # => Callback Extension
-        extend ActiveModel::Callbacks
-        define_model_callbacks :initialize, only: :after
-
-        # => Initialize
-        # => http://api.rubyonrails.org/classes/ActiveModel/Callbacks.html
-        # => http://stackoverflow.com/a/17682228/1143732
-        def initialize attributes={}
-          super
-          run_callbacks :initialize do
-            # => Needed for after_initialize
-          end
-        end
-
-      end
-
-      ##################################
-      ##################################
-
-      ####################
-      #      Table       #
-      ####################
-
-          # Schema
-          ###################
-          # class_name      @exception.class.name
-          # status          ActionDispatch::ExceptionWrapper.new(@request.env, @exception).status_code
-          # message         @exception.message
-          # trace           @exception.backtrace.join("\n")
-          # target          @request.url
-          # referer         @request.referer
-          # params          @request.params.inspect
-          # user_agent      @request.user_agent
-          # created_at
-          # updated_at
-
-        # => Table is called "errors"
-        # => Dev needs to use migration to create db
-        if ExceptionHandler.config.try(:db)
-          def self.table_name
-            ExceptionHandler.config.db
-          end
-        end
-
-      ##################################
-      ##################################
-
-        ####################
-        #     Options      #
-        ####################
-
-        # => Email
-        # => after_initialize invoked after .new method called
-        # => Should have been after_create but user may not save
-        after_initialize Proc.new { |e| ExceptionHandler::ExceptionMailer.new_exception(e).deliver } if ExceptionHandler.config.try(:email).try(:is_a?, String)
-
-        # => Attributes
-        attr_accessor :request, :klass, :exception, :description
-        attr_accessor *ATTRS unless ExceptionHandler.config.try(:db)
-
-        # => Validations
-        validates :klass, exclusion:    { in: [ActionController::RoutingError, AbstractController::ActionNotFound, ActiveRecord::RecordNotFound], message: "%{value}" }, if: "referer.blank?"
-        validates :user_agent, format:  { without: Regexp.new( BOTS.join("|"), Regexp::IGNORECASE ) }
-
-      ##################################
-      ##################################
-
-        ####################################
-        # Virtual
-        ####################################
-
-          # => Klass
-          # => Used for validation (needs to be cleaned up in 0.7.0)
-          def klass
-            exception.class
-          end
-
-          # => Exception (virtual)
-          def exception
-            request.env['action_dispatch.exception']
-          end
-
-          # => Description
-          def description
-            I18n.with_options scope: [:exception_handler], message: message, status: status do |i18n|
-              i18n.t response, default: Rack::Utils::HTTP_STATUS_CODES[status] || status
-            end
-          end
-
-        ####################################
-        # Exception
-        ####################################
-
-          # => Class Name
-          def class_name
-            exception.class.name
-          end
-
-          # => Message
-          def message
-            exception.message
-          end
-
-          # => Trace
-          def trace
-            exception.backtrace.join("\n")
-          end
-
-        ####################################
-        # Request
-        ####################################
-
-          # => Target URL
-          def target
-            request.url
-          end
-
-          # => Referrer URL
-          def referer
-            request.referer
-          end
-
-          # => Params
-          def params
-            request.params.inspect
-          end
-
-          # => User Agent
-          def user_agent
-            request.user_agent
-          end
-
-        ####################################
-        # Other
-        ####################################
-
-          # => Status code (404, 500 etc)
-          def status
-            ActionDispatch::ExceptionWrapper.new(request.env, exception).status_code
-          end
-
-          # => Server Response ("Not Found" etc)
-          def response
-            ActionDispatch::ExceptionWrapper.rescue_responses[class_name]
-          end
-
-      ##################################
-      ##################################
-
-    end
-  end
-
-  ############################################################
-  ############################################################
+module ExceptionHandler
+
+  ############################################################
+  ############################################################
+
+    # => Search Bots
+    # => Used in "Exception" class
+    BOTS = %w(Baidu Gigabot Googlebot libwww-per lwp-trivial msnbot SiteUptime Slurp Wordpress ZIBB ZyBorg Yandex Jyxobot Huaweisymantecspider ApptusBot)
+
+    # => Attributes
+    # => Determine schema etc
+    ATTRS = %i(class_name status message trace target referrer params user_agent)
+
+  ############################################################
+  ############################################################
+
+    # => Class (inheritance dependent on whether db option is available)
+    self::Exception =
+      Class.new( (ExceptionHandler.config.try(:db) && defined?(ActiveRecord)) ? ActiveRecord::Base : Object ) do
+
+      # => Include individual elements
+      # => Only required if no db present (no ActiveRecord)
+      if ExceptionHandler.config.try(:db)
+
+        # => Set Attrs
+        def initialize attributes={}
+            super
+            ATTRS.each do |type|
+              self[type] = eval type.to_s
+            end
+        end
+
+      else
+
+        # => ActiveModel
+        include ActiveModel::Model
+        include ActiveModel::Validations
+
+        # => Callback Extension
+        extend ActiveModel::Callbacks
+        define_model_callbacks :initialize, only: :after
+
+        # => Initialize
+        # => http://api.rubyonrails.org/classes/ActiveModel/Callbacks.html
+        # => http://stackoverflow.com/a/17682228/1143732
+        def initialize attributes={}
+          super
+          run_callbacks :initialize do
+            # => Needed for after_initialize
+          end
+        end
+
+      end
+
+      ##################################
+      ##################################
+
+      ####################
+      #      Table       #
+      ####################
+
+          # Schema
+          ###################
+          # class_name      @exception.class.name
+          # status          ActionDispatch::ExceptionWrapper.new(@request.env, @exception).status_code
+          # message         @exception.message
+          # trace           @exception.backtrace.join("\n")
+          # target          @request.url
+          # referer         @request.referer
+          # params          @request.params.inspect
+          # user_agent      @request.user_agent
+          # created_at
+          # updated_at
+
+        # => Table is called "errors"
+        # => Dev needs to use migration to create db
+        if ExceptionHandler.config.try(:db)
+          def self.table_name
+            ExceptionHandler.config.db
+          end
+        end
+
+      ##################################
+      ##################################
+
+        ####################
+        #     Options      #
+        ####################
+
+        # => Email
+        # => after_initialize invoked after .new method called
+        # => Should have been after_create but user may not save
+        after_initialize -> (e) { ExceptionHandler::ExceptionMailer.new_exception(e).deliver }, if: :email? # => see bottom of file
+
+        # => Attributes
+        attr_accessor :request, :klass, :exception, :description
+        attr_accessor *ATTRS unless ExceptionHandler.config.try(:db)
+
+        # => Validations
+        validates :user_agent, format: { without: Regexp.new( BOTS.join("|"), Regexp::IGNORECASE ) }
+
+      ##################################
+      ##################################
+
+        ####################################
+        # Virtual
+        ####################################
+
+          # => Exception (virtual)
+          # => Basis on which all the class is built
+          def exception
+            request.env['action_dispatch.exception']
+          end
+
+          # => Klass
+          # => Used for validation (needs to be cleaned up in 0.7.0)
+          def klass
+            exception.class
+          end
+
+          # => Description
+          def description
+            I18n.with_options scope: [:exception_handler], message: message, status: status do |i18n|
+              i18n.t response, default: Rack::Utils::HTTP_STATUS_CODES[status]
+            end
+          end
+
+        ####################################
+        # Exception
+        ####################################
+
+          # => Class Name
+          def class_name
+            exception.class.name
+          end
+
+          # => Message
+          def message
+            exception ? exception.message : Rack::Utils::HTTP_STATUS_CODES[status]
+          end
+
+          # => Trace
+          def trace
+            exception.backtrace.join("\n")
+          end
+
+        ####################################
+        # Request
+        ####################################
+
+          # => Target URL
+          def target
+            request.url
+          end
+
+          # => Referrer URL
+          def referer
+            request.referer
+          end
+
+          # => Params
+          def params
+            request.params.inspect
+          end
+
+          # => User Agent
+          def user_agent
+            request.user_agent
+          end
+
+        ####################################
+        # Other
+        ####################################
+
+          # => Status code (404, 500 etc)
+          def status
+            exception ? ActionDispatch::ExceptionWrapper.new(request.env, exception).try(:status_code) : request.env["PATH_INFO"][1..-1].to_i
+          end
+
+          # => Server Response ("Not Found" etc)
+          def response
+            ActionDispatch::ExceptionWrapper.rescue_responses[class_name]
+          end
+
+      ##################################
+      ##################################
+
+      private
+
+        # => Email
+        # => should be on the same line as after_initialize but too long
+        def email?
+          ExceptionHandler.config.try(:email).try(:is_a?, String) && ExceptionHandler.config.options(status, :notification) != false
+        end
+
+    end
+  end
+
+  ############################################################
+  ############################################################

data/app/views/exception_handler/exceptions/show.html.erb

@@ -1,3 +1,3 @@
-<%= content_tag :div, class: "exception", data: { status: @exception.status, response: @exception.response.to_s.humanize, rails: Rails.version }, onclick: ("location.href=\"#{root_url}\";" if @exception.status == "500" && Rails.application.routes.recognize_path("/")), title: ("Return Home" if @exception.status == "500" && Rails.application.routes.recognize_path("/")) do %>
-  <%= content_tag :span, @exception.description.html_safe %>
-<% end %>
+<%= content_tag :div, class: "exception", data: { status: @exception.status, response: @exception.response.to_s.humanize, rails: Rails.version }, onclick: ("location.href=\"#{root_url}\";" if @exception.status.to_s.first == "5" && Rails.application.routes.recognize_path("/")) do %>
+  <%= content_tag :span, @exception.description.html_safe %>
+<% end %>

data/app/views/exception_handler/mailers/layout.haml

@@ -1,8 +1,8 @@
-!!!
-%html
-  %head
-    %meta{:content => "text/html; charset=utf-8", "http-equiv" => "Content-Type"}/
-    :css
-      /* Email styles need to be inline */
-  %body
-    = yield
+!!!
+%html
+  %head
+    %meta{:content => "text/html; charset=utf-8", "http-equiv" => "Content-Type"}/
+    :css
+      /* Email styles need to be inline */
+  %body
+    = yield