errdo 0.12.5 → 0.12.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: fa5c0d97349694ebbeceb83f7caade8e98d720b9
-  data.tar.gz: fc76d6ae232506190db13610073fed132b7fe4a6
+  metadata.gz: 1e93e649455c8417dd8748fc01cfcdd19323d02c
+  data.tar.gz: 59616d8f89381572ae7366029d6b44fee0da8135
 SHA512:
-  metadata.gz: 7e94c13be169ea3792b59e864e00e8b94eac22d0da3f0704cb4834c62f90974368ad6006f302a59fdc66a1c387a55cb7696cc2209b319eafb608ec655885d804
-  data.tar.gz: 0391eba0b3218e8770866a1fdb34ead125e116d3169d55e333a8dd9e2300bd360766f8c827c169c0d845a6fab040a397f8b376407504fbdd131942e8abd4b98f
+  metadata.gz: e47e6476ce20e9c925800af27c321599674da8f152cb35c3e44c1b5f8b547ffbdd11412a033573746bf110ad0da5287f2fad90d8f829e2bbc31133652d2a90ba
+  data.tar.gz: 3e4370ec0fc36bd8dd5ed433cf8f9f4bfcc88b6c43ee7dc25dc908cb8ae65be17607de56693eaea10d41534e4239ebf25b3a2bf1618d6dc35ed3768ff5e899dd

data/app/models/errdo/error.rb

@@ -28,9 +28,11 @@ module Errdo
 
     # I need a more elegant way to do this
     def self.create_unique_string_from_params(params)
-      params[:backtrace][0].to_s.last(50) +
-        params[:exception_message].to_s.last(20) +
-        params[:exception_class_name].to_s.last(20)
+      Digest::SHA1.hexdigest(
+        params[:backtrace].reject { |l| l[%r{\/ruby-[0-9]\.[0-9]\.[0-9]\/|_test.rb}] }.join("") +
+        params[:exception_message].to_s.gsub(/:0x[0-f]{14}/, "") +
+        params[:exception_class_name].to_s
+      ).to_s
     end
 
     def short_backtrace
@@ -47,9 +49,10 @@ module Errdo
     private
 
     def create_unique_string
-      self.backtrace_hash = backtrace.to_a[0].to_s.last(50) +
-                            exception_message.to_s.last(20) +
-                            exception_class_name.to_s.last(20)
+      self.backtrace_hash =
+        Digest::SHA1.hexdigest(backtrace.to_a.reject { |l| l[%r{\/ruby-[0-9]\.[0-9]\.[0-9]\/|_test.rb}] }.join("") +
+                               exception_message.to_s.gsub(/:0x[0-f]{14}/, "") +
+                               exception_class_name.to_s).to_s
     end
 
   end

data/app/views/errdo/errors/show.html.slim

@@ -39,7 +39,7 @@
           = user_show_string(@occurrence.experiencer)
 
       h3 Param Values
-      div == @occurrence.param_values
+      div = @occurrence.param_values
 
       h3 Backtrace
       .errdo-backtrace-box

data/lib/errdo/version.rb

@@ -1,3 +1,3 @@
 module Errdo
-  VERSION = '0.12.5'.freeze
+  VERSION = '0.12.6'.freeze
 end

data/test/dummy/app/controllers/static_controller.rb

@@ -11,6 +11,11 @@ class StaticController < ApplicationController
     raise "e" * 256
   end
 
+  def deep_error
+    eval(params[:code])
+    render :home
+  end
+
   def log
     Errdo.error "This is the error", user: current_user, data: "This is the data"
   end

data/test/dummy/config/routes.rb

@@ -3,6 +3,7 @@ Rails.application.routes.draw do
   get 'static/home'
   get 'static/generic_error'
   get 'static/long_error'
+  get 'static/deep_error'
   get 'static/log'
   get 'static/view_error'
   get 'static/double_render_error'