exception-track 0.6.0 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d153785e412ef19575e9fa1c1c09efd6e4c6641ca6b7ef0f2874ed8c48ded09a
-  data.tar.gz: ec5ffb28de5d0e292edcd088156163158ace9068487759df0aaaf8e658595c90
+  metadata.gz: 032c9ba1ad0589f704bedbff866d3bab2aae870abc44251b2e8b5f504732fe77
+  data.tar.gz: 72946a5e0284dac9fd318671aa46f2322c10522010a6d2c076024801d7749be9
 SHA512:
-  metadata.gz: 2570f322779022af1c78361dff981300b7967c0fb4fa1710be7571945fc56431d9c54bf2a2ac9aa0a5d8d02d7e6c351ac6ced157d9dd992d43d4396da1a4cc4c
-  data.tar.gz: 32544198077f5147a86adafdd65f5dad1a71c82f893bb80a9ed2f2bb432f2ffd081e277cbe580ae4b10c7ab44097537a9f66021eff4368dbc721f77aa203232c
+  metadata.gz: 4e2fe6f4660aec15140e9fa70f1876a51d78e1c61fdc98cfadfaaee08a5426fc3ddba66e6b3b858cc119293add6503a95bd3934de779bdcb22cb23ad0bfbfe61
+  data.tar.gz: 3807967a319b57e6e7a1fe1dc0647e3bbb4496729b47ddd392d9a36973f4d76c7470dfb646c085a841d9921b5295e8e6e6de170534a55d51bcd739295cdebbb6

data/README.md

@@ -1,5 +1,4 @@
-ExceptionTrack
---------------
+## ExceptionTrack
 
 Tracking exceptions for Rails application store them in database.
 
@@ -7,7 +6,7 @@ This gem is base on [exception_notification](https://github.com/smartinez87/exce
 
 [中文介绍和使用说明](https://ruby-china.org/topics/32325)
 
-[![Gem Version](https://badge.fury.io/rb/exception-track.svg)](https://badge.fury.io/rb/exception-track) [![Build Status](https://travis-ci.org/rails-engine/exception-track.svg)](https://travis-ci.org/rails-engine/exception-track) [![Code Climate](https://codeclimate.com/github/rails-engine/exception-track/badges/gpa.svg)](https://codeclimate.com/github/rails-engine/exception-track) [![codecov.io](https://codecov.io/github/rails-engine/exception-track/coverage.svg?branch=master)](https://codecov.io/github/rails-engine/exception-track?branch=master) [![](http://inch-ci.org/github/rails-engine/exception-track.svg?branch=master)](http://inch-ci.org/github/rails-engine/exception-track?branch=master)
+[![Gem Version](https://badge.fury.io/rb/exception-track.svg)](https://badge.fury.io/rb/exception-track) [![build](https://github.com/rails-engine/exception-track/workflows/build/badge.svg)](https://github.com/rails-engine/exception-track/actions?query=workflow%3Abuild) [![codecov.io](https://codecov.io/github/rails-engine/exception-track/coverage.svg?branch=master)](https://codecov.io/github/rails-engine/exception-track?branch=master)
 
 ![2017-02-17 12 35 18](https://cloud.githubusercontent.com/assets/5518/23052599/8e267c02-f50d-11e6-8d6e-cef0cc1991b7.png)
 
@@ -41,6 +40,12 @@ Rails.application.routes.draw do
 end
 ```
 
+Migrate database:
+
+```shell
+bundle exec rails db:migrate
+```
+
 Now you can open: http://localhost:3000/exception-track
 
 ## Configuration

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

@@ -1,27 +1,78 @@
+:root {
+  --body-background-color: #eee;
+  --card-background-color: #fff;
+  --card-border-color: #fff;
+  --card-shadow-color: rgba(200, 200, 200, 0.26);
+  --text-color: #333;
+  --text-color1: #666;
+  --text-color2: #999;
+  --link-color: #364cc9;
+  --table-background-color1: #f5f5f5;
+  --table-border-color: #f0f0f0;
+  --input-border-color: #ddd;
+  --button-background-color: #fff;
+  --button-border-color: #eee;
+  --button-text-color: #555;
+  --button-hover-background-color: #f7f7f7;
+  --button-hover-border-color: #eee;
+  --button-danger-color: #f85149;
+  --button-danger-hover-background-color: #f85149;
+  --button-danger-hover-color: #fff;
+  --pre-background-color: #f7f7f7;
+  --notice-color: #fff;
+  --notice-background-color: #3CBD46;
+}
+
+@media (prefers-color-scheme: dark) {
+  :root {
+    --body-background-color: #0d1117;
+    --card-background-color: rgba(200, 200, 200, 0.02);
+    --card-border-color: #30363d;
+    --card-shadow-color: rgba(0, 0, 0, 0.16);
+    --text-color: #c9d1d9;
+    --text-color1: #999;
+    --text-color2: #888;
+    --link-color: #58a6ff;
+    --table-background-color1: #161b22;
+    --table-border-color: #30363d;
+    --input-border-color: #333;
+    --button-background-color: #21262d;
+    --button-border-color: #30363d;
+    --button-text-color: #c9d1d9;
+    --button-hover-background-color: #30363d;
+    --button-hover-border-color: #8b949e;
+    --pre-background-color: #161b22;
+    --notice-color: #3CBD46;
+    --notice-background-color: #3cbd4621;
+  }
+}
 
 body {
   margin: 0;
   padding: 20px 0;
-  background-color: #eee;
+  background-color: var(--body-background-color);
 }
 
 body, textarea {
   font-family: -apple-system, BlinkMacSystemFont, "Helvetica Neue", Arial, Helvetica, sans-serif;
   font-size: 14px;
   line-height: 1.4;
-  color: #333;
+  color: var(--text-color);
 }
 
 .footer {
   padding: 15px;
   text-align: center;
-  color: #999;
+  color: var(--text-color2);
+}
+
+.footer a:link, .footer a:visited {
+  color: var(--text-color1);
+  text-decoration: underline;
 }
-.footer a:link,
-.footer a:visited { color: #666; text-decoration: underline;}
 
 a, a:visited, a:active {
-  color: #364cc9;
+  color: var(--link-color);
   text-decoration: none;
 }
 
@@ -34,15 +85,15 @@ table {
   border-collapse: collapse;
   border-spacing: 0;
   margin-bottom: 20px;
+  border: 1px solid var(--table-border-color);
+  border-radius: 3px;
 }
 
-th {
-  text-align: left;
-  border-bottom: solid 1px #e0e0e0;
+td.date {
+  width: 170px;
+  font-size: 12px;
 }
 
-td.date { width: 150px; font-size: 12px; }
-
 h1 {
   margin-top: 0;
   font-size: 20px;
@@ -66,29 +117,28 @@ ul {
 table td, table th {
   padding: 10px 15px;
 }
-th { background: #f5f5f5; border-bottom: 1px solid #e0e0e0; }
-td {
-  border-top: solid 1px #e0e0e0;
+
+th {
+  text-align: left;
+  background: var(--table-background-color1);
+  border-bottom: 1px solid var(--table-border-color);
 }
 
-pre {
-  background-color: #eee;
-  padding: 10px;
-  white-space: pre-wrap;
-  word-break: break-word;
+td {
+  border-top: solid 1px var(--table-border-color);
 }
 
 textarea {
   width: 100%;
   height: 100px;
-  border: solid 1px #ddd;
+  border: solid 1px var(--input-border-color);
   padding: 10px;
 }
 
 hr {
   border: none;
   height: 0;
-  border-top: solid 1px #ddd;
+  border-top: solid 1px var(--input-border-color);
   margin-bottom: 15px;
 }
 
@@ -107,16 +157,29 @@ hr {
   -ms-user-select: none;
   user-select: none;
   border-radius: 3px;
-  border: 1px solid #ccc;
+  transition: .2s cubic-bezier(.3, 0, .5, 1);
+  transition-property: color, background-color, border-color;
+  border: 1px solid var(--button-border-color);
   padding: 6px 16px;
-  color: #555 !important;
+  color: var(--button-text-color) !important;
   outline: 0 !important;
-  background: #FFF;
+  background: var(--button-background-color);
 }
-.btn:hover { text-decoration: none !important; background: #f7f7f7; }
-.btn-danger { background: #fff; color: #E33F00 !important; border-color: #E33F00;}
+
+.btn:hover {
+  text-decoration: none !important;
+  background: var(--button-hover-background-color);
+}
+
+.btn-danger {
+  background: var(--button-background-color);
+  color: var(--button-danger-color) !important;
+  border-color: var(--button-border-color);
+}
+
 .btn-danger:hover {
-  background: #FCEDEC;
+  background: var(--button-danger-hover-background-color);
+  color: var(--button-danger-hover-color) !important;
 }
 
 .container {
@@ -124,8 +187,9 @@ hr {
   margin-left: auto;
   margin-right: auto;
   padding: 20px;
-  background-color: #fff;
-  box-shadow: 0 1px 8px rgba(200, 200, 200, 0.26);
+  background-color: var(--card-background-color);
+  box-shadow: 0 1px 8px var(--card-shadow-color);
+  border: 1px solid var(--card-border-color);
   border-radius: 3px;
 }
 
@@ -140,49 +204,67 @@ hr {
   height: 34px;
   line-height: 34px;
 }
-.toolbar form { display: inline; }
-.toolbar .pull-right { float: right; }
+
+.toolbar form {
+  display: inline;
+}
+
+.toolbar .pull-right {
+  float: right;
+}
 
 #notice {
   padding: 8px 15px;
-  background: #3CBD46;
-  color: #fff;
+  background: var(--notice-background-color);
+  color: var(--notice-color);
   margin-bottom: 15px;
   border-radius: 3px;
 }
 
 pre {
-  background: #f7f7f7;
+  white-space: pre-wrap;
+  word-break: break-word;
+  background: var(--pre-background-color);
   padding: 25px;
   border-radius: 3px;
   font-size: 12px;
   font-family: Menlo, Monaco, Consolas, monospace;
 }
 
-h1 { font-size: 16px; }
+h1 {
+  font-size: 16px;
+}
 
 .pagination {
   padding-bottom: 15px;
   font-size: 14px;
 }
 
-.pagination li { display: inline; }
+.pagination li {
+  display: inline;
+}
 
 .pagination a {
   display: inline-block;
   padding: 5px 10px;
-  border: 1px solid #eee;
-  color: #555;
+  border-radius: 3px;
+  border: 1px solid var(--button-border-color);
+  color: var(--button-text-color);
+  background: var(--button-background-color);
   text-decoration: none;
 }
+
 .pagination a:hover {
-  background: #f7f7f7;
+  background: var(--button-hover-background-color);
+  color: var(--button-hover-color);
+  border-color: var(--button-hover-border-color);
 }
-.pagination em,
-.pagination .current {
+
+.pagination em, .pagination .current {
+  border-radius: 3px;
   display: inline-block;
   padding: 5px 10px;
-  border: 1px solid #f0f0f0;
-  background: #f0f0f0;
+  border: 1px solid var(--button-hover-border-color);
+  background: var(--button-hover-background-color);
   font-style: normal;
-}
+}

data/app/controllers/exception_track/logs_controller.rb

@@ -3,7 +3,7 @@
 module ExceptionTrack
   class LogsController < ActionController::Base
     layout "exception-track/application"
-    before_action :set_log, only: %i[show destroy]
+    before_action :set_log, only: :show
 
     # GET /exception_logs
     def index
@@ -12,7 +12,7 @@ module ExceptionTrack
 
     def export
       @logs = Log.order("id desc").where("created_at >= ?", 3.months.ago)
-      send_data JSON.pretty_generate(@logs.as_json(only: [:title, :body, :created_at, :updated_at])), filename: "#{Date.current}.json", disposition: "attachment"
+      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
@@ -25,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

@@ -6,7 +6,7 @@
   }
 </script>
 <div class="toolbar">
-  <%= @logs.total_count %> exceptions
+  <a class="btn" href="/">Back to App</a>
 
   <div class="pull-right">
     <%= form_tag(export_logs_path, method: 'post') do %>
@@ -21,6 +21,7 @@
 <% if @logs.blank? %>
   <div class="no-record">No exceptions.</div>
 <% else %>
+  <h1><%= pluralize(@logs.total_count, 'exception') %></h1>
   <%= paginate @logs %>
 
   <table class="table table-borded tabl">

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/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,13 +15,14 @@ module ExceptionTrack
   class << self
     def config
       return @config if defined?(@config)
+
       @config = Configuration.new
       @config.environments = %i[development production]
       @config
     end
 
     def configure(&block)
-      config.instance_exec(&block)
+      config.instance_eval(&block)
     end
   end
 end

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.6.0"
+  VERSION = "1.2.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)
@@ -43,9 +47,7 @@ module ExceptionNotifier
       headers = []
       headers << "Method:      #{env['REQUEST_METHOD']}"
       headers << "URL:         #{env['REQUEST_URI']}"
-      if env['REQUEST_METHOD'].downcase != "get"
-        headers << "Parameters:\n#{pretty_hash(parameters.except(:controller, :action), 13)}"
-      end
+      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']}"
@@ -59,11 +61,11 @@ module ExceptionNotifier
 
     def filter_parameters(env)
       parameters = env["action_dispatch.request.parameters"] || {}
-      parameter_filter = ActionDispatch::Http::ParameterFilter.new(env["action_dispatch.parameter_filter"] || [])
-      return parameter_filter.filter(parameters)
-    rescue => e
+      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}"
-      return parameters
+      parameters
     end
 
     def pretty_hash(params, indent = 0)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: exception-track
 version: !ruby/object:Gem::Version
-  version: 0.6.0
+  version: 1.2.0
 platform: ruby
 authors:
 - Jason Lee
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-01-31 00:00:00.000000000 Z
+date: 2020-12-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: exception_notification
@@ -44,28 +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'
-- !ruby/object:Gem::Dependency
-  name: mysql2
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
+        version: '5.2'
 description: Tracking exceptions for Rails application store them in database by exception_notification
   gem.
 email:
@@ -89,6 +75,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
@@ -96,7 +83,7 @@ homepage: https://github.com/rails-engine/exception-track
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -111,8 +98,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.1
-signing_key: 
+rubygems_version: 3.1.4
+signing_key:
 specification_version: 4
 summary: Tracking exceptions for Rails application store them in database.
 test_files: []