crashbreak 0.9.5 → 0.9.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ccf2481e1c5893e9ed41bbcec7a57adade15abf1
-  data.tar.gz: 91e98a300ed88abeef023e169df88cbc89fe1b62
+  metadata.gz: d943a7d2f3a78353805545a04922acdf8e841d79
+  data.tar.gz: 5eb0864882ebfcc111be7bfd9e2639b0e449fd67
 SHA512:
-  metadata.gz: 8759aee358aa1273aee4f3a08f06a0c25712fcd93b851ac19606deebf5aacaa606c41dcb846e8ac41125f3b0f74c603484c24a49a3681d60b15a13e1e4253b85
-  data.tar.gz: 24453d50056e35ae6705564eb20d77a2360da515860cb59b10d103e537710eb2bb1655243603e4684d038fc8c842aef04fe265eb0d2f1acd15c8d0dee9c6ce1c
+  metadata.gz: 801a7c44cc6ea929bc54aeecfb0b89b45989c91006a487d215e9b46b7b9d56cf7b9dcff13c1175d326bb763be0a63e99ab3392dad323830aff1b035302dab90b
+  data.tar.gz: 23cc67630df928d9b638a758563c7300115d4eb1bafd507375dada7f1bc2bdf53419603ce97b12249e9150606e9399f9493618bc567495e9435a45bd1c981979

data/crashbreak.gemspec

@@ -28,4 +28,6 @@ Gem::Specification.new do |spec|
   spec.add_runtime_dependency 'request_store', '>= 0'
   spec.add_runtime_dependency 'octokit', '>= 0'
   spec.add_runtime_dependency 'aws-sdk', '~> 2'
+  spec.add_runtime_dependency 'sidekiq', '>= 0'
+  spec.add_runtime_dependency 'oj', '>= 0'
 end

data/lib/crashbreak.rb

@@ -3,6 +3,9 @@ require 'faraday'
 require 'request_store'
 require 'octokit'
 require 'aws-sdk'
+require 'sidekiq'
+require 'oj'
+
 require 'crashbreak/version'
 require 'crashbreak/exception_notifier'
 require 'crashbreak/formatters/basic_formatter'
@@ -19,6 +22,8 @@ require 'crashbreak/dumpers_data_repository'
 require 'crashbreak/request_parser'
 require 'crashbreak/github_integration_service'
 require 'crashbreak/AWS'
+require 'crashbreak/async_exception_notifier'
+require 'crashbreak/tiny_exception_notifier'
 
 require 'dumpers/database_dumper'
 require 'dumpers/request_dumper'

data/lib/crashbreak/async_exception_notifier.rb

@@ -0,0 +1,16 @@
+module Crashbreak
+  class AsyncExceptionNotifier
+    include Sidekiq::Worker
+
+    def perform(request_store_data)
+      RequestStore.store.merge!(Oj.load(request_store_data))
+      exception_notifier.notify
+    end
+
+    private
+
+    def exception_notifier
+      @exception_notifier ||= ExceptionNotifier.new
+    end
+  end
+end

data/lib/crashbreak/exception_catcher_middleware.rb

@@ -11,13 +11,40 @@ module Crashbreak
         RequestStore.store[:exception] = exception
         store_variables_from_env env
 
-        exception_notifier.notify
+        notify_about_exception
         raise
       end
     end
 
     private
 
+    def notify_about_exception
+      if exception_notifier.respond_to?(:perform_async)
+        prepare_request_store_for_serialization
+        exception_notifier.perform_async Oj.dump(RequestStore.store)
+      else
+        exception_notifier.notify
+      end
+    end
+
+    def prepare_request_store_for_serialization
+      request_hash = {}
+
+      RequestStore.store[:request].env.each do |key, value|
+        request_hash[key] = value.to_s
+      end
+
+      RequestStore.store[:request] = request(request_hash)
+
+      controller = RequestStore.store[:controller].dup
+
+      [:@_env, :@_request, :@_response, :@_lookup_context].each do |variable_symbol|
+        controller.instance_variable_set(variable_symbol, nil)
+      end
+
+      RequestStore.store[:controller] = controller
+    end
+
     def exception_notifier
       Crashbreak.configure.exception_notifier
     end

data/lib/crashbreak/tiny_exception_notifier.rb

@@ -0,0 +1,18 @@
+module Crashbreak
+  class TinyExceptionNotifier < ExceptionNotifier
+    def notify(exception = RequestStore.store[:exception])
+      RequestStore.store[:exception] = exception
+      exceptions_repository.create serialize_exception
+    end
+
+    private
+
+    def serializers
+      [BasicInformationFormatter.new]
+    end
+
+    def dumpers
+      []
+    end
+  end
+end

data/lib/crashbreak/version.rb

@@ -1,3 +1,3 @@
 module Crashbreak
-  VERSION = '0.9.5'
+  VERSION = '0.9.6'
 end

data/lib/tasks/crashbreak.rake

@@ -5,7 +5,10 @@ namespace :crashbreak do
     begin
       raise CrashbreakTestError.new('If you see this message everything works fine!')
     rescue CrashbreakTestError => error
-      Crashbreak.configure.exception_notifier.notify error
+      RequestStore.store[:exception] = error
+      RequestStore.store[:request] = Rack::Request.new({})
+
+      Crashbreak.configure.exception_notifier.notify
     end
 
     puts 'Done, now check if error exists on crashbreak.com!'

data/spec/lib/async_exception_notifier_spec.rb

@@ -0,0 +1,19 @@
+describe Crashbreak::AsyncExceptionNotifier do
+  subject { described_class.new }
+  let(:request_data) { Oj.dump(Hash[request: 'request_info', current_user: 'current_user_data']) }
+
+  before(:each) do
+    expect_any_instance_of(Crashbreak::ExceptionNotifier).to receive(:notify).and_return(true)
+  end
+
+  it 'calls notify on exception notifier' do
+    subject.perform(request_data)
+  end
+
+  it 'sets request_data in RequestStore' do
+    expect(RequestStore.store[:request]).to eq('request_info')
+    expect(RequestStore.store[:current_user]).to eq('current_user_data')
+
+    subject.perform(request_data)
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: crashbreak
 version: !ruby/object:Gem::Version
-  version: 0.9.5
+  version: 0.9.6
 platform: ruby
 authors:
 - Michal Janeczek
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-04-10 00:00:00.000000000 Z
+date: 2015-04-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -164,6 +164,34 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '2'
+- !ruby/object:Gem::Dependency
+  name: sidekiq
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: oj
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Maybe later... :)
 email:
 - michal.janeczek@ymail.com
@@ -181,6 +209,7 @@ files:
 - crashbreak.gemspec
 - lib/crashbreak.rb
 - lib/crashbreak/AWS.rb
+- lib/crashbreak/async_exception_notifier.rb
 - lib/crashbreak/config/configurable.rb
 - lib/crashbreak/config/configurator.rb
 - lib/crashbreak/dumpers_data_repository.rb
@@ -195,6 +224,7 @@ files:
 - lib/crashbreak/formatters/summary_formatter.rb
 - lib/crashbreak/github_integration_service.rb
 - lib/crashbreak/request_parser.rb
+- lib/crashbreak/tiny_exception_notifier.rb
 - lib/crashbreak/version.rb
 - lib/dumpers/database_dumper.rb
 - lib/dumpers/request_dumper.rb
@@ -208,6 +238,7 @@ files:
 - lib/tasks/crashbreak.rake
 - spec/dumpers/request_dumper_spec.rb
 - spec/features/sending_error_report_spec.rb
+- spec/lib/async_exception_notifier_spec.rb
 - spec/lib/config/configurable_spec.rb
 - spec/lib/config/configurator_spec.rb
 - spec/lib/dumpers_data_repository_spec.rb
@@ -254,6 +285,7 @@ summary: Take a break from crashes!
 test_files:
 - spec/dumpers/request_dumper_spec.rb
 - spec/features/sending_error_report_spec.rb
+- spec/lib/async_exception_notifier_spec.rb
 - spec/lib/config/configurable_spec.rb
 - spec/lib/config/configurator_spec.rb
 - spec/lib/dumpers_data_repository_spec.rb