exception_hunter 0.1.1 → 0.4.2

data/app/views/exception_hunter/errors/_error_user_data.erb

@@ -3,9 +3,8 @@
     Unfortunately, no user information has been registered for this error.
   </div>
 <% else %>
-  <% error.user_data.transform_keys(&:to_sym).each do |key, value| %>
-    <div class="column column-15">
-      <b><%= key %></b>: <%= value %>
-    </div>
-  <% end %>
+  <pre class="tracked-data">
+
+<%= format_tracked_data(error.user_data) %>
+  </pre>
 <% end %>

data/app/views/exception_hunter/errors/_errors_table.erb

@@ -0,0 +1 @@
+<%= render partial: 'exception_hunter/errors/error_row', collection: errors, as: :error %>

data/app/views/exception_hunter/errors/_last_7_days_errors_table.erb

@@ -0,0 +1,12 @@
+<% today_errors = errors.select { |error| error.show_for_day?(Date.current) } %>
+<%= render partial: 'exception_hunter/errors/error_row', collection: today_errors, as: :error %>
+
+<% yesterday_errors = errors.select { |error| error.show_for_day?(Date.yesterday) } %>
+<div class="errors-date-group">Yesterday</div>
+<%= render partial: 'exception_hunter/errors/error_row', collection: yesterday_errors, as: :error %>
+
+<% (2..6).each do |i| %>
+  <% errors_on_day = errors.select { |error| error.show_for_day?(i.days.ago) } %>
+  <div class="errors-date-group"><%= ExceptionHunter::ErrorGroupPresenter.format_occurrence_day(i.days.ago) %></div>
+  <%= render partial: 'exception_hunter/errors/error_row', collection: errors_on_day, as: :error %>
+<% end %>

data/app/views/exception_hunter/errors/index.html.erb

@@ -1,36 +1,77 @@
-<div class="row statistics-row">
-  <div class="column column-offset-50 column-25">
-    <div class="statistics__cell">
-      <%= number_with_delimiter(@errors_count) %> Total errors
-    </div>
-  </div>
-  <div class="column column-25">
-    <div class="statistics__cell">
-      <%= number_with_delimiter(@month_errors) %> Errors this month
+<div class="row">
+  <div class="column column-80">
+    <div class="errors-tabs">
+      <div class="errors-tab <%= 'errors-tab--active' if @dashboard.tab_active?(@dashboard.class::LAST_7_DAYS_TAB) %>">
+        <%= link_to errors_path(tab: @dashboard.class::LAST_7_DAYS_TAB) do %>
+          <div class="errors-tab__content">
+            <div class="errors-tab__badge">
+              <%= @dashboard.errors_count(@dashboard.class::LAST_7_DAYS_TAB) %>
+            </div>
+            <div class="errors-tab__description">
+              Errors in the last 7 days
+            </div>
+          </div>
+        <% end %>
+      </div>
+
+      <div class="errors-tab <%= 'errors-tab--active' if @dashboard.tab_active?(@dashboard.class::CURRENT_MONTH_TAB) %>">
+        <%= link_to errors_path(tab: @dashboard.class::CURRENT_MONTH_TAB) do %>
+          <div class="errors-tab__content">
+            <div class="errors-tab__badge">
+              <%= @dashboard.errors_count(@dashboard.class::CURRENT_MONTH_TAB) %>
+            </div>
+            <div class="errors-tab__description">
+              Errors this month
+            </div>
+          </div>
+        <% end %>
+      </div>
+
+      <div class="errors-tab <%= 'errors-tab--active' if @dashboard.tab_active?(@dashboard.class::TOTAL_ERRORS_TAB) %>">
+        <%= link_to errors_path(tab: @dashboard.class::TOTAL_ERRORS_TAB) do %>
+          <div class="errors-tab__content">
+            <div class="errors-tab__badge">
+              <%= @dashboard.errors_count(@dashboard.class::TOTAL_ERRORS_TAB) %>
+            </div>
+            <div class="errors-tab__description">
+              Total errors
+            </div>
+          </div>
+        <% end %>
+      </div>
+
+      <div class="errors-tab errors-tab__resolved <%= 'errors-tab--active' if @dashboard.tab_active?(@dashboard.class::RESOLVED_ERRORS_TAB) %>">
+        <%= link_to errors_path(tab: @dashboard.class::RESOLVED_ERRORS_TAB) do %>
+          <div class="errors-tab__content">
+            <div class="errors-tab__badge">
+              <%= @dashboard.errors_count(@dashboard.class::RESOLVED_ERRORS_TAB) || '-' %>
+            </div>
+            <div class="errors-tab__description">
+              Resolved
+            </div>
+          </div>
+        <% end %>
+      </div>
     </div>
   </div>
-</div>
 
-<div class="row error-row error-row--header">
-  <div class="column column-75">Message</div>
-  <div class="column column-15">Last Occurrence</div>
-  <div class="column column-10">Occurrences</div>
+  <div class="column column-10 column-offset-10">
+    <%= button_to 'Purge', purge_errors_path,
+                  class: %w[button purge-button],
+                  method: :delete,
+                  data: { confirm: 'This will delete all stale errors, do you want to continue?' } %>
+  </div>
 </div>
 
-<% @errors.each do |error| %>
-  <div class="row error-row">
-    <div class="column column-75 error-cell">
-      <%= link_to error.message, error_path(error.id) %>
-    </div>
-    <div class="column column-15 error-cell error-cell--highlight">
-      <% if error.last_occurrence.present? %>
-        <%= time_ago_in_words(error.last_occurrence) %> ago
-      <% else %>
-        Never
-      <% end %>
-    </div>
-    <div class="column column-10 error-cell error-cell--highlight">
-      <%= error.total_occurrences %>
-    </div>
+<div class="errors__container">
+  <div class="row error-row error-row--header">
+    <div class="column column-10">Tags</div>
+    <div class="column column-40">Message</div>
+    <div class="column column-15">First Occurrence</div>
+    <div class="column column-15">Last Occurrence</div>
+    <div class="column column-10">Total</div>
+    <div class="column column-10"></div>
   </div>
-<% end %>
+
+  <%= render partial: @dashboard.partial_for_tab, locals: { errors: @errors } %>
+</div>

data/app/views/exception_hunter/errors/pagy/_pagy_nav.html.erb

@@ -1,17 +1,17 @@
-<% link = pagy_link_proc(pagy) %>
-<nav aria-label="pager" class="pagy_nav pagination" role="navigation">
-  <% if pagy.prev %>
-  <span class="page prev"><%== link.call(pagy.prev, '◄', 'aria-label="previous"') %></span>
-  <% else %>
-    <span class="page prev disabled">
-      ◄
-    </span>
+<div class="error-occurrences__nav">
+  <%= link_to pagy_url_for(1, pagy) do %>
+    <%= button_tag 'First', class: %w[button button-outline error-occurrences__nav-link], disabled: pagy.prev.nil? %>
   <% end %>
-  <% if pagy.next %>
-    <span class="page next"><%== link.call(pagy.next, '►', 'aria-label="next"') %></span>
-  <% else %>
-    <span class="page next disabled">
-      ►
-    </span>
+  <%= link_to pagy_url_for(pagy.prev, pagy) do %>
+    <%= button_tag '<', class: %w[button button-outline error-occurrences__nav-link], disabled: pagy.prev.nil? %>
   <% end %>
-</nav>
+  <div class="error-occurrences__nav-current">
+    <%= occurred_at %> (<%= pagy.page %>/<%= pagy.last %>)
+  </div>
+  <%= link_to pagy_url_for(pagy.next, pagy) do %>
+    <%= button_tag '>', class: %w[button button-outline error-occurrences__nav-link], disabled: pagy.next.nil? %>
+  <% end %>
+  <%= link_to pagy_url_for(pagy.last, pagy) do %>
+    <%= button_tag 'Last', class: %w[button button-outline error-occurrences__nav-link], disabled: pagy.next.nil? %>
+  <% end %>
+</div>

data/app/views/exception_hunter/errors/show.html.erb

@@ -1,33 +1,69 @@
-<div class="row">
-  <div class="column column-offset-80 column-20">
-    <%= render partial: 'exception_hunter/errors/pagy/pagy_nav', locals: { pagy: @pagy } %>
+<div class="row error-occurrence__header">
+  <div class="column column-10">
+    <% @error.tags.each do|tag| %>
+    <div class="error-cell__tags">
+      <span class="error-tag"><%= tag %></span>
+    </div>
+    <% end %>
+  </div>
+  <div class="column column-80">
+    <div class="error-title">
+      <%= @error.class_name %>: <%= @error.message %>
+    </div>
+  </div>
+  <div class="column column-10">
+    <%= button_to('Resolve', resolved_errors_path(error_group: { id: @error.error_group_id }),
+                  method: :post,
+                  class: %w[button resolve-button],
+                  data: { confirm: 'Are you sure you want to resolve this error?' }) %>
   </div>
 </div>
 
-<div class="error-title">
-  <%= @error.class_name %>: <%= @error.message %>
-</div>
-<div class="error-occurred_at">
-  <%= @error.occurred_at %>
+<div class="row">
+  <div class="column column-50">
+    <ul data-tabs class="errors-tabs">
+      <li class="errors-tab">
+        <a data-tabby-default href="#summary">
+          <div class="errors-tab__content">
+            Summary
+          </div>
+        </a>
+      </li>
+      <li class="errors-tab">
+        <a href="#backtrace">
+          <div class="errors-tab__content">
+            Backtrace
+          </div>
+        </a>
+      </li>
+      <li class="errors-tab">
+        <a href="#user-data">
+          <div class="errors-tab__content">
+            User Data
+          </div>
+        </a>
+      </li>
+    </ul>
+  </div>
+  <div class="column column-50">
+    <%= render partial: 'exception_hunter/errors/pagy/pagy_nav', locals: { pagy: @pagy, occurred_at: @error.occurred_at } %>
+  </div>
 </div>
 
-<ul data-tabs>
-  <li><a data-tabby-default href="#summary">Summary</a></li>
-  <li><a href="#backtrace">Backtrace</a></li>
-  <li><a href="#user-data">User Data</a></li>
-</ul>
 
-<div class="tab-content">
-  <div id="summary">
-    <%= render partial: 'exception_hunter/errors/error_summary', locals: { error: @error } %>
-  </div>
+<div class="errors__container">
+  <div class="tab-content">
+    <div id="summary">
+      <%= render partial: 'exception_hunter/errors/error_summary', locals: { error: @error } %>
+    </div>
 
-  <div id="backtrace">
-    <%= render partial: 'exception_hunter/errors/error_backtrace', locals: { error: @error } %>
-  </div>
+    <div id="backtrace">
+      <%= render partial: 'exception_hunter/errors/error_backtrace', locals: { error: @error } %>
+    </div>
 
-  <div id="user-data">
-    <%= render partial: 'exception_hunter/errors/error_user_data', locals: { error: @error } %>
+    <div id="user-data">
+      <%= render partial: 'exception_hunter/errors/error_user_data', locals: { error: @error } %>
+    </div>
   </div>
 </div>
 

data/app/views/layouts/exception_hunter/application.html.erb

@@ -7,12 +7,11 @@
 
   <%= favicon_link_tag 'exception_hunter/logo.png' %>
 
-  <link rel="stylesheet" href="//fonts.googleapis.com/css?family=Roboto:300,300italic,700,700italic">
+  <link rel="stylesheet" href="https://rsms.me/inter/inter.css">
   <link rel="stylesheet" href="//cdnjs.cloudflare.com/ajax/libs/normalize/5.0.0/normalize.css">
   <link rel="stylesheet" href="//cdnjs.cloudflare.com/ajax/libs/milligram/1.3.0/milligram.css">
 
   <!-- Get patch fixes within a minor version -->
-  <link rel="stylesheet" type="text/css" href="https://cdn.jsdelivr.net/gh/cferdinandi/tabby@12.0/dist/css/tabby-ui.min.css">
   <script src="https://cdn.jsdelivr.net/gh/cferdinandi/tabby@12.0/dist/js/tabby.polyfills.min.js"></script>
 
   <%= stylesheet_link_tag "exception_hunter/application", media: "all" %>
@@ -22,18 +21,78 @@
 <div class="wrapper">
   <nav class="nav">
     <div class="container container__nav">
-      <div class="nav__logo">
-        <%= link_to errors_path do %>
-          <%= image_tag 'exception_hunter/logo.png', alt: 'Logo' %>
-        <% end %>
+      <div class="row">
+        <div class="column column-40">
+          <%= link_to errors_path do %>
+            <div class="nav__title">Exception Hunter</div>
+          <% end %>
+        </div>
+        <div class="column column-10 column-offset-50">
+          <div class="logout">
+            <% if current_admin_user? %>
+              <div class="sign_out">
+                <%= link_to exception_hunter_logout_path do %>
+                  Sign out
+                <% end %>
+              </div>
+            <% end %>
+          </div>
+        </div>
       </div>
     </div>
   </nav>
 
   <div class="container">
+    <% if flash[:notice] %>
+      <div id="flash-notice" class="flash flash--notice row">
+        <div class="column column-80">
+          <%= flash[:notice] %>
+        </div>
+        <div class="column column-20">
+          <div class="button button-clear button-dismiss" data-dismiss="#flash-notice">
+            <%= ['Cool!', 'Nice!', 'Ok', 'Dismiss', 'Fine. Whatever.', 'I know that'].sample %>
+          </div>
+        </div>
+      </div>
+    <% end %>
+
     <%= yield %>
   </div>
+
+  <div class="footer">
+    Made with ♥ at <a href="https://www.rootstrap.com/" class="text--underline">Rootstrap</a>
+  </div>
 </div>
 
+<script type="text/javascript" charset="utf-8">
+  function confirmable(element) {
+    const message = element.getAttribute('data-confirm')
+    element.form.addEventListener('submit', confirm(message))
+
+    function confirm(message) {
+      return function (event) {
+        if (!window.confirm(message)) {
+          event.preventDefault()
+        }
+      }
+    }
+  }
+
+  function dismissible(element) {
+    element.addEventListener('click', dismiss(element))
+
+    function dismiss(element) {
+      return function () {
+        const selector = element.getAttribute('data-dismiss')
+        const elementToDismiss = document.querySelector(selector)
+        elementToDismiss.remove()
+      }
+    }
+  }
+
+  Array.from(document.querySelectorAll('[data-confirm]')).forEach(confirmable)
+  Array.from(document.querySelectorAll('[data-dismiss]')).forEach(dismissible)
+</script>
+
 </body>
 </html>

data/app/views/layouts/exception_hunter/exception_hunter_logged_out.html.erb

@@ -0,0 +1,24 @@
+<!DOCTYPE html>
+<html>
+<head>
+  <title>Exception Hunter</title>
+  <%= csrf_meta_tags %>
+  <%= csp_meta_tag %>
+
+  <link rel="stylesheet" href="//fonts.googleapis.com/css?family=Roboto:300,300italic,700,700italic">
+  <link rel="stylesheet" href="//cdnjs.cloudflare.com/ajax/libs/normalize/5.0.0/normalize.css">
+  <link rel="stylesheet" href="//cdnjs.cloudflare.com/ajax/libs/milligram/1.3.0/milligram.css">
+  <link rel="stylesheet" href="https://rsms.me/inter/inter.css">
+
+  <!-- Get patch fixes within a minor version -->
+  <link rel="stylesheet" type="text/css" href="https://cdn.jsdelivr.net/gh/cferdinandi/tabby@12.0/dist/css/tabby-ui.min.css">
+  <script src="https://cdn.jsdelivr.net/gh/cferdinandi/tabby@12.0/dist/js/tabby.polyfills.min.js"></script>
+
+  <%= stylesheet_link_tag "exception_hunter/application", media: "all" %>
+</head>
+<body style="background-color: #E5E5E5;">
+<div class="container">
+  <%= yield %>
+</div>
+</body>
+</html>

data/config/rails_best_practices.yml

@@ -15,9 +15,8 @@ MoveCodeIntoModelCheck: { use_count: 2 }
 MoveFinderToNamedScopeCheck: { }
 MoveModelLogicIntoModelCheck: { use_count: 4 }
 NeedlessDeepNestingCheck: { nested_count: 2 }
-NotRescueExceptionCheck: { ignored_files: 'request_hunter.rb' }
 NotUseDefaultRouteCheck: { }
-NotUseTimeAgoInWordsCheck: { ignored_files: ['index.html.erb'] }
+#NotUseTimeAgoInWordsCheck: { ignored_files: ['index.html.erb'] }
 OveruseRouteCustomizationsCheck: { customize_count: 3 }
 ProtectMassAssignmentCheck: { }
 RemoveEmptyHelpersCheck: { }
@@ -30,7 +29,7 @@ ReplaceComplexCreationWithFactoryMethodCheck: { attribute_assignment_count: 2 }
 ReplaceInstanceVariableWithLocalVariableCheck: { }
 RestrictAutoGeneratedRoutesCheck: { }
 SimplifyRenderInControllersCheck: { }
-SimplifyRenderInViewsCheck: { }
+#SimplifyRenderInViewsCheck: { }
 #UseBeforeFilterCheck: { customize_count: 2 }
 UseModelAssociationCheck: { }
 UseMultipartAlternativeAsContentTypeOfEmailCheck: { }

data/config/routes.rb

@@ -1,3 +1,21 @@
 ExceptionHunter::Engine.routes.draw do
-  resources :errors, only: %i[index show]
+  resources :errors, only: %i[index show] do
+    delete 'purge', on: :collection, to: 'errors#destroy', as: :purge
+  end
+
+  resources :resolved_errors, only: %i[create]
+
+  get '/', to: redirect('/exception_hunter/errors')
+
+  if ExceptionHunter::Config.auth_enabled?
+    admin_user_class = ExceptionHunter::Config.admin_user_class.underscore.to_sym
+
+    devise_scope admin_user_class do
+      get '/login', to: 'devise/sessions#new', as: :exception_hunter_login
+      post '/login', to: 'devise/sessions#create', as: :exception_hunter_create_session
+      get '/logout', to: 'devise/sessions#destroy', as: :exception_hunter_logout
+    end
+
+    devise_for admin_user_class, only: []
+  end
 end

data/lib/exception_hunter.rb

@@ -1,15 +1,25 @@
+require 'pagy'
+
 require 'exception_hunter/engine'
-require 'exception_hunter/railtie'
+require 'exception_hunter/middleware/request_hunter'
 require 'exception_hunter/config'
+require 'exception_hunter/error_creator'
+require 'exception_hunter/error_reaper'
+require 'exception_hunter/tracking'
 require 'exception_hunter/user_attributes_collector'
-require 'pagy'
 
 module ExceptionHunter
+  autoload :Devise, 'exception_hunter/devise'
+
+  extend ::ExceptionHunter::Tracking
+
   def self.setup(&block)
     block.call(Config)
   end
 
   def self.routes(router)
+    return unless Config.enabled
+
     router.mount(ExceptionHunter::Engine, at: 'exception_hunter')
   end
 end