exception-track 0.5.2 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f4e53ae1240f7be104a8b3616737f5927b3a350a27555e15b8ac9b016c9b5300
-  data.tar.gz: 96536183c297c9c861d195f1c0720d1abf3a72ba81a9344f223930d2cb026f04
+  metadata.gz: 204adb921c2d193136a838e4b3612e80162b59eda35b0e6c245c529038d98d88
+  data.tar.gz: eb21c43240e595356f99c4790112120f337625d4ba9da0359fe30dbb6f9fc774
 SHA512:
-  metadata.gz: 861a62df66eb3e662550deb9e0a73b7e4cef0bce0b33e46008ea18477df20532ec4967b3b7c708eda73cf7dab4ed995ee4f7c82269f297288e67037eb02ea3f8
-  data.tar.gz: 3bef483c5f518d65fb836dd2dc196ec8a649a8b58db309925778f3b1dec6ca296a4700da16b92f8ec1f60984eeef8f0c18f5b4fbd25f2868653bf16e26d7b2b2
+  metadata.gz: 350a1ba8b5ec9836b93c38cb40f68083fbb484a7c7435a39025068cafb50eeb6def491f187c3c29cba24f0fd5ead757c3f435fcad896c2ce49e2c05420657675
+  data.tar.gz: c602e2dd3ba3bd8f5bef4bee20c4910c72733e8557d14831751a46b55ecbcaf918de7a96c7d492c7dcc144147a474758105cd715c95ce2c4ce4900ab963e746d

data/app/assets/config/exception_track_manifest.js

@@ -1,2 +0,0 @@
-//= link_directory ../javascripts/exception-track.js
-//= link_directory ../stylesheets/exception-track.css

data/app/assets/stylesheets/exception-track/application.css

@@ -6,7 +6,7 @@ body {
 }
 
 body, textarea {
-  font-family: "Helvetica Neue", Arial, Helvetica, sans-serif;
+  font-family: -apple-system, BlinkMacSystemFont, "Helvetica Neue", Arial, Helvetica, sans-serif;
   font-size: 14px;
   line-height: 1.4;
   color: #333;
@@ -38,7 +38,7 @@ table {
 
 th {
   text-align: left;
-  border-bottom: solid 1px #ddd;
+  border-bottom: solid 1px #e0e0e0;
 }
 
 td.date { width: 150px; font-size: 12px; }
@@ -66,9 +66,9 @@ ul {
 table td, table th {
   padding: 10px 15px;
 }
-th { background: #f5f5f5; border-bottom: 1px solid #eee; }
+th { background: #f5f5f5; border-bottom: 1px solid #e0e0e0; }
 td {
-  border-top: solid 1px #ddd;
+  border-top: solid 1px #e0e0e0;
 }
 
 pre {
@@ -108,12 +108,16 @@ hr {
   user-select: none;
   border-radius: 3px;
   border: 1px solid #ccc;
-  padding: 8px 18px;
+  padding: 6px 16px;
+  color: #555 !important;
   outline: 0 !important;
   background: #FFF;
 }
-.btn:hover { text-decoration: none; }
-.btn-danger { background: #E25517; color: #fff; border: #D44323;}
+.btn:hover { text-decoration: none !important; background: #f7f7f7; }
+.btn-danger { background: #fff; color: #E33F00 !important; border-color: #E33F00;}
+.btn-danger:hover {
+  background: #FCEDEC;
+}
 
 .container {
   max-width: 1000px;
@@ -121,6 +125,8 @@ hr {
   margin-right: auto;
   padding: 20px;
   background-color: #fff;
+  box-shadow: 0 1px 8px rgba(200, 200, 200, 0.26);
+  border-radius: 3px;
 }
 
 .no-record {
@@ -134,6 +140,7 @@ hr {
   height: 34px;
   line-height: 34px;
 }
+.toolbar form { display: inline; }
 .toolbar .pull-right { float: right; }
 
 #notice {
@@ -145,9 +152,9 @@ hr {
 }
 
 pre {
-  background: #f9f9f9;
-  padding: 10px;
-  border: 1px solid #eee;
+  background: #f7f7f7;
+  padding: 25px;
+  border-radius: 3px;
   font-size: 12px;
   font-family: Menlo, Monaco, Consolas, monospace;
 }
@@ -165,6 +172,11 @@ h1 { font-size: 16px; }
   display: inline-block;
   padding: 5px 10px;
   border: 1px solid #eee;
+  color: #555;
+  text-decoration: none;
+}
+.pagination a:hover {
+  background: #f7f7f7;
 }
 .pagination em,
 .pagination .current {

data/app/controllers/exception_track/logs_controller.rb

@@ -10,6 +10,11 @@ module ExceptionTrack
       @logs = Log.order("id desc").page(params[:page]).per(15)
     end
 
+    def export
+      @logs = Log.order("id desc").where("created_at >= ?", 3.months.ago)
+      send_data JSON.pretty_generate(@logs.as_json(only: %i[title body created_at updated_at])), filename: "#{Date.current}.json", disposition: "attachment"
+    end
+
     # GET /exception_logs/1
     def show; end
 
@@ -20,7 +25,6 @@ module ExceptionTrack
     end
 
     private
-
       # Use callbacks to share common setup or constraints between actions.
       def set_log
         @log = Log.find(params[:id])

data/app/views/exception_track/logs/index.html.erb

@@ -9,6 +9,9 @@
   <%= @logs.total_count %> exceptions
 
   <div class="pull-right">
+    <%= form_tag(export_logs_path, method: 'post') do %>
+      <button type="submit" class="btn">Export last 3 months</button>
+    <% end %>
     <%= form_tag(all_logs_path, method: 'delete') do %>
       <button type="submit" class="btn btn-danger" onclick="return confirmDestroyAll()">Delete All</button>
     <% end %>

data/config/initializers/exception-track.rb

@@ -9,10 +9,15 @@ ExceptionTrack.configure do
 end
 
 # ExceptionNotification.configure do |config|
-#   config.ignored_exceptions += %w(ActionView::TemplateError
-#                                   ActionController::InvalidAuthenticityToken
-#                                   ActionController::BadRequest
-#                                   ActionView::MissingTemplate
-#                                   ActionController::UrlGenerationError
-#                                   ActionController::UnknownFormat)
+#   config.ignored_exceptions += %w[
+#     ActionView::TemplateError
+#     ActionController::InvalidAuthenticityToken
+#     ActionController::BadRequest
+#     ActionView::MissingTemplate
+#     ActionController::UrlGenerationError
+#     ActionController::UnknownFormat
+#     ActionController::InvalidCrossOriginRequest
+#     ActionController::ParameterMissing
+#     Mime::Type::InvalidMimeType
+#   ]
 # end

data/config/routes.rb

@@ -4,6 +4,7 @@ ExceptionTrack::Engine.routes.draw do
   resources :logs, path: "" do
     collection do
       delete :all
+      post :export
     end
   end
 end

data/db/migrate/20170217023900_create_exception_track_logs.rb

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
-class CreateExceptionTrackLogs < ActiveRecord::Migration[5.0]
+class CreateExceptionTrackLogs < ActiveRecord::Migration[5.2]
   def change
     create_table :exception_tracks do |t|
       t.string :title
-      t.text :body, limit: 16777215
+      t.text :body, limit: 16_777_215
 
       t.timestamps
     end

data/lib/exception-track.rb

@@ -2,6 +2,7 @@
 
 require "exception-track/version"
 require "exception-track/configuration"
+require "exception-track/log_subscriber"
 require "exception-track/engine"
 
 require "exception_notification"
@@ -14,6 +15,7 @@ module ExceptionTrack
   class << self
     def config
       return @config if defined?(@config)
+
       @config = Configuration.new
       @config.environments = %i[development production]
       @config

data/lib/exception-track/configuration.rb

@@ -8,6 +8,7 @@ module ExceptionTrack
 
     def enabled_env?(env)
       return false if env.blank?
+
       environments.include?(env.to_sym)
     end
   end

data/lib/exception-track/engine.rb

@@ -3,5 +3,11 @@
 module ExceptionTrack
   class Engine < ::Rails::Engine
     isolate_namespace ExceptionTrack
+
+    initializer "exception-track.assets.precompile", group: :all do |app|
+      app.config.assets.precompile += %w[exception-track/application.css]
+    end
+
+    ExceptionTrack::LogSubscriber.attach_to :exception_track
   end
 end

data/lib/exception-track/log_subscriber.rb

@@ -0,0 +1,12 @@
+# frozen_string_literal: true
+
+module ExceptionTrack
+  class LogSubscriber < ActiveSupport::LogSubscriber
+    # ActiveSupport::Notifications.instrument('track.exception_track', action: action)
+    def track(event)
+      prefix = color("ExceptionTrack", CYAN)
+      title = color(event.payload[:title], RED)
+      debug "  #{prefix} track db (#{event.duration.round(1)}ms)"
+    end
+  end
+end

data/lib/exception-track/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ExceptionTrack
-  VERSION = "0.5.2"
+  VERSION = "1.1.0"
 end

data/lib/exception_notifier/exception_track_notifier.rb

@@ -2,27 +2,31 @@
 
 module ExceptionNotifier
   class ExceptionTrackNotifier < ExceptionNotifier::BaseNotifier
-    def initialize(_options); end
+    def initialize(_opts); end
 
-    def call(exception, _options = {})
+    def call(exception, opts = {})
       return unless ExceptionTrack.config.enabled_env?(Rails.env)
 
       # send the notification
       title = exception.message || "None"
-
       messages = []
-      messages << headers_for_env(_options[:env])
-      messages << ""
-      messages << "--------------------------------------------------"
-      messages << ""
-      messages << exception.inspect
-      unless exception.backtrace.blank?
-        messages << "\n"
-        messages << exception.backtrace
-      end
 
-      ExceptionTrack::Log.create(title: title[0, 200], body: messages.join("\n"))
-    rescue => e
+      ActiveSupport::Notifications.instrument("track.exception_track", title: title) do
+        messages << headers_for_env(opts[:env])
+        messages << ""
+        messages << "--------------------------------------------------"
+        messages << ""
+        messages << exception.inspect
+        unless exception.backtrace.blank?
+          messages << "\n"
+          messages << exception.backtrace
+        end
+
+        Rails.logger.silence do
+          ExceptionTrack::Log.create(title: title[0, 200], body: messages.join("\n"))
+        end
+      end
+    rescue StandardError => e
       errs = []
       errs << "-- [ExceptionTrack] create error ---------------------------"
       errs << e.message.indent(2)
@@ -38,11 +42,12 @@ module ExceptionNotifier
     def headers_for_env(env)
       return "" if env.blank?
 
-      parameters = env["action_dispatch.request.parameters"] || {}
+      parameters = filter_parameters(env)
 
       headers = []
       headers << "Method:      #{env['REQUEST_METHOD']}"
       headers << "URL:         #{env['REQUEST_URI']}"
+      headers << "Parameters:\n#{pretty_hash(parameters.except(:controller, :action), 13)}" if env["REQUEST_METHOD"].downcase != "get"
       headers << "Controller:  #{parameters['controller']}##{parameters['action']}"
       headers << "RequestId:   #{env['action_dispatch.request_id']}"
       headers << "User-Agent:  #{env['HTTP_USER_AGENT']}"
@@ -53,5 +58,19 @@ module ExceptionNotifier
 
       headers.join("\n")
     end
+
+    def filter_parameters(env)
+      parameters = env["action_dispatch.request.parameters"] || {}
+      parameter_filter = ActiveSupport::ParameterFilter.new(env["action_dispatch.parameter_filter"] || [])
+      parameter_filter.filter(parameters)
+    rescue StandardError => e
+      Rails.logger.error "filter_parameters error: #{e.inspect}"
+      parameters
+    end
+
+    def pretty_hash(params, indent = 0)
+      json = JSON.pretty_generate(params)
+      json.indent(indent)
+    end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: exception-track
 version: !ruby/object:Gem::Version
-  version: 0.5.2
+  version: 1.1.0
 platform: ruby
 authors:
 - Jason Lee
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-01-19 00:00:00.000000000 Z
+date: 2020-06-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: exception_notification
@@ -44,14 +44,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '5.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '5.2'
 - !ruby/object:Gem::Dependency
   name: mysql2
   requirement: !ruby/object:Gem::Requirement
@@ -89,6 +89,7 @@ files:
 - lib/exception-track.rb
 - lib/exception-track/configuration.rb
 - lib/exception-track/engine.rb
+- lib/exception-track/log_subscriber.rb
 - lib/exception-track/version.rb
 - lib/exception_notifier/exception_track_notifier.rb
 - lib/generators/exception_track/install_generator.rb
@@ -111,7 +112,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.1
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: Tracking exceptions for Rails application store them in database.