global_error_handler 0.1.2 → 0.1.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8fdb1574c39aa350dde0f8e6be54526e65940069
-  data.tar.gz: 2d3c783156410adb2b5b77029083920fda393f12
+  metadata.gz: c8e1d5697e1bfb783ebc6f2ea2ba9e90d476c148
+  data.tar.gz: 1d6f01e6a420998b71837234e4f36246866d4f9e
 SHA512:
-  metadata.gz: ac108f47e998a5daccfc3b0665fe8df3fd78358baf5928a44b35ef73ab17fcbaa859de1337bf8626ed657907a1f33203910a645bab591c0e3177164c29298097
-  data.tar.gz: 140d3aa099996dcbdff08ba5fc5444a62d49d2490cd09f3416071e643d107812ba540857a6df5996cacbd6d43a0ee6404bd182c26fd117f66e4f2dcd68a34fbf
+  metadata.gz: 048a744a25f0bdc8c9b96a7d68358af9e9d0862d5cf70a1ded5c69a7dc823cedf9ab727ea9530a72aa904ab828fc1fe19eeef67ed82c6a85eb369f19db0ae696
+  data.tar.gz: 9d5a3260b8daac46772ced2e7f1ebf20101fa5d64134b288a76453189647ca1196ef7d96e42566e0c326654143df18925d1cdc51e3d8966f9530ca390bd16bbf

data/README.md

@@ -9,6 +9,7 @@ It adds Exceptions tab to Redis Web server in case to view, filter, delete or tr
   - [Exceptions tab](#exceptions-tab)
   - [Truncate / Delete](#truncatedelete-functionality)
   - [RescueFrom](#rescue_from)
+  - [Subscribe to Redis notifications](#subscribe-to-redis-notifications)
 - [Data structure](#data-structure)
 - [Contributing](#contributing)
 
@@ -55,7 +56,7 @@ If `rescue_from` is used in your application, add following line at top of the m
 GlobalErrorHandler::Handler.new(request.env, exception).process_exception!
 ```
 
-### Subscribe to expired key notifications
+### Subscribe to Redis notifications
 Following command should run on your server in case to automatically clear filters on deleting keys from redis due to expiration.
 Default expiration time is set to 4 weeks (`GlobalErrorHandler::Redis::REDIS_TTL`)
 

data/config/recipes/global_error_handler.rb

@@ -0,0 +1,19 @@
+namespace :global_error_handler do
+  desc 'Subscribe to expiration'
+  task :subscribe do
+    run %Q{cd #{latest_release} && RAILS_ENV=#{rails_env} nohup rake global_error_handler:subscribe_to_expired >/dev/null 2>&1 & sleep 2}
+  end
+
+  desc 'Unsubscribe from expiration'
+  task :unsubscribe do
+    run %Q{cd #{latest_release} && #{rake} RAILS_ENV=#{rails_env} global_error_handler:unsubscribe_from_expired}
+  end
+
+  desc 'Update Subscription to expiration'
+  task :update_subscription do
+    unsubscribe
+    subscribe
+  end
+  after 'deploy:restart', 'global_error_handler:update_subscription'
+end
+

data/lib/global_error_handler/redis.rb

@@ -9,6 +9,7 @@ class GlobalErrorHandler::Redis
 
   class << self
     def store(info_hash)
+      return if info_hash.blank?
       redis_key = exception_key(next_id!)
       redis.hmset redis_key, info_hash.merge(id: current_id).to_a.flatten
       redis.rpush EXCEPTIONS_REDIS_KEY, redis_key
@@ -18,12 +19,14 @@ class GlobalErrorHandler::Redis
       redis.expire redis_key, REDIS_TTL
     end
 
+    def initialize_redis_from_config #:nodoc:
+      redis_config = YAML.load_file(File.join(Rails.root, 'config', 'redis.yml'))[Rails.env]
+      Redis.new(redis_config['global_exception_handler'])
+    end
+
     def redis
       @redis ||= begin
-                   unless $redis_global_exception_handler.is_a? Redis
-                     redis_config = YAML.load_file(File.join(Rails.root, 'config', 'redis.yml'))[Rails.env]
-                     $redis_global_exception_handler = Redis.new(redis_config['global_exception_handler'])
-                   end
+                   $redis_global_exception_handler = initialize_redis_from_config unless $redis_global_exception_handler.is_a? Redis
                    $redis_global_exception_handler
                  end
     end
@@ -92,10 +95,46 @@ class GlobalErrorHandler::Redis
 
     def clear_filters(key)
       FILTER_FIELDS.each do |field|
+        retry_count = 0
         field_value = build_filter_value(redis.hget key, field)
-        filter_keys_for(field, field_value).each do |filter_key|
-          redis.lrem filter_key, 1, key
+        begin
+          filter_keys_for(field, field_value).each do |filter_key|
+            redis.lrem filter_key, 1, key
+          end
+        rescue
+          field_value = ''
+          retry if (retry_count += 1) < 2
+        end
+      end
+    end
+
+    def cleanup_database_dependencies!
+      total_exceptions_count = exceptions_count
+      total_exception_keys_count = redis.keys(exception_key('*')).size
+      if total_exceptions_count > total_exception_keys_count
+        puts "==> Database dependency is broken. Need to fix it!"
+        start = 0
+        per_page = 500
+        exception_keys_to_be_cleaned_up = []
+        valid_chunks_count = 0
+        cleanup_count = exception_keys_to_be_cleaned_up.size
+        while total_exceptions_count > start
+          exception_keys(start, per_page).each do |redis_key|
+            exception_keys_to_be_cleaned_up.push redis_key unless redis.exists(redis_key)
+          end
+          if cleanup_count == (cleanup_count = exception_keys_to_be_cleaned_up.size)
+            valid_chunks_count += 1
+          end
+          break if valid_chunks_count > 3 #if three ranges in a row are consistent, treat database consistency and finish looping
+          start += per_page
+        end
+
+        puts "*** found #{exception_keys_to_be_cleaned_up.count} broken dependency keys."
+        exception_keys_to_be_cleaned_up.each do |redis_key|
+          delete_dependencies(redis_key) rescue next
         end
+      else
+        puts "==> Database dependency is OK. No need to fix it!"
       end
     end
 

data/lib/global_error_handler/redis_notification_subscriber.rb

@@ -41,7 +41,7 @@ class GlobalErrorHandler::RedisNotificationSubscriber
     end
 
     def redis
-      GlobalErrorHandler::Redis.redis
+      @redis ||= GlobalErrorHandler::Redis.initialize_redis_from_config
     end
 
     def sub_channel

data/lib/global_error_handler/version.rb

@@ -1,3 +1,3 @@
 module GlobalErrorHandler
-  VERSION = "0.1.2"
+  VERSION = "0.1.10"
 end

data/lib/tasks/global_error_handler.rake

@@ -11,7 +11,7 @@ namespace :global_error_handler do
   end
 
   desc 'Unsubscribe from expired keyevent notifications'
-  task :unsubscribe_from_expired do
+  task unsubscribe_from_expired: :environment do
     puts '*** pid file does not exist!' or next unless File.exists?(pid_file)
     process_id = File.read(pid_file).to_i
     begin
@@ -29,6 +29,13 @@ namespace :global_error_handler do
     end
   end
 
+  desc 'Clean database dependencies for exception keys'
+  task cleanup_database_dependencies: :environment do
+    puts '** starting CleanUp process...'
+    GlobalErrorHandler::Redis.cleanup_database_dependencies!
+    puts '** completed CleanUp process.'
+  end
+
   def pid_file
     @pid_file ||= File.expand_path('./tmp/pids/global_error_handler_subscription.pid')
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: global_error_handler
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.10
 platform: ruby
 authors:
 - Andrii Rudenko
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-12-26 00:00:00.000000000 Z
+date: 2015-03-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -97,6 +97,7 @@ files:
 - LICENSE.txt
 - README.md
 - Rakefile
+- config/recipes/global_error_handler.rb
 - config/redis_example.yml
 - global_error_handler.gemspec
 - lib/global_error_handler.rb