bellman 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c3fbd1ee19a3727dfb7efd00bafd2de608cc08c7cecffbaa311a2f0ad71c51d9
-  data.tar.gz: 853afcf30bffdbbf08d6b81192725543650ddad687e6ed8645fa70082887f645
+  metadata.gz: c65c02d5ddc7dc46aba77a42fc162c5f35b77bba264a990cc5c66580fd5bd397
+  data.tar.gz: 484d19f9b7546a31599afef4184ecff5a9ef43e24bd6dd46294c532a02707815
 SHA512:
-  metadata.gz: caf2124a78105f6eae4e0bb82599febb623411405bb4c87ec15e001ceeb57b400655e42330c657020f29031992be1327b17995b6aea484e75251e46a08621542
-  data.tar.gz: 770d99680d228da731033197bc7108b905904b3a56865c5f4ce5c514be950c71b8c235393e444dbce09590cad60b014b3bf4a3203ecedc7f2cdb86195cf6d5a6
+  metadata.gz: '03491319b209b2994b3329c12aafa4aba3ea3b09bb3046aff6cfc4d0899797748f591384ddc3b6e2df713a00bc7e6921c1b44ef1aa7e2c64d79d6d8a4bf2e2bb'
+  data.tar.gz: 80961f465408e2880701fc8f41f9a21157fa989da5c6d2231f8c53b891bea76ede01177d464c3eaa3962191a028b16e9168cd1f34e30f0de62abfc02ed435918

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    bellman (0.1.1)
+    bellman (0.1.2)
       activerecord (> 4.2.0)
       activesupport (> 4.2.0)
 

data/README.md

@@ -81,6 +81,12 @@ For example, here is a common configuration for when an application uses the Rai
 ```ruby
 Bellman.configure do |config|
   config.default_severity = :error
+
+  # Anything other than :raise will set the severity level to the default
+  # severity if an unkown one is past. By default Bellman will raise an error
+  # so that you can catch your mistakes in development
+  config.on_unknown_severity = :raise  
+  
   config.handlers = [
       {
         id: :log,

data/lib/bellman/bellman.rb

@@ -19,6 +19,7 @@ module Bellman
     # Set the defaults
     config.severities = %i[debug info warn error fatal].freeze
     config.default_severity = :error
+    config.on_unknown_severity = :raise
     config.handlers = [
       {
         id: :log,
@@ -38,12 +39,12 @@ module Bellman
     @handler = Handlers::Handler.new
   end
 
-  def self.handler
-    @handler
-  end
-
-  def self.error_handler(id:)
-    @handler.handlers.find { |hndlr| hndlr[:id] == id.to_sym }
+  def self.handler(id: nil)
+    if id.nil?
+      @handler
+    else
+      @handler.handlers.find { |hndlr| hndlr[:id] == id.to_sym }
+    end
   end
 
   # rubocop:disable Metrics/ParameterLists

data/lib/bellman/handlers/handler.rb

@@ -14,10 +14,8 @@ module Bellman
       # rubocop:disable Metrics/ParameterLists
       def handle(error, severity: :error, trace_id: nil, objects: nil,
                  data: nil, include_backtrace: false, handlers: nil)
-        unless Bellman.config.severities.include?(severity)
-          severity = Bellman.config.default_severity
-        end
 
+        severity = handle_severity(severity)
         handlers = if handlers.present?
                      process_handlers_config(handlers)
                    else
@@ -43,6 +41,16 @@ module Bellman
 
       private
 
+      def handle_severity(severity)
+        return severity if Bellman.config.severities.include?(severity)
+        if Bellman.config.on_unknown_severity != :raise
+          return Bellman.config.default_severity
+        end
+
+        raise "Unknown severity level #{severity}. " \
+              "Must be one of #{Bellman.config.severities}"
+      end
+
       def process_handlers_config(configs)
         configs.map { |config| process_handler_config(config) }
       end
@@ -68,7 +76,7 @@ module Bellman
                                config[:class].new(**config[:params])
                              end
         elsif config[:id]
-          config = Bellman.error_handler(config[:id])
+          config = Bellman.handler(config[:id])
           config[:handler] = if config[:params].blank?
                                config[:class].new
                              else
@@ -88,12 +96,12 @@ module Bellman
 
         case config
         when Symbol
-          handler = Bellman.error_handler(id: config)
+          handler = Bellman.handler(id: config)
           return handler if handler
         when String
           # If the string matches the ID of a handler defined in the Bellman
           # default config, return that.
-          handler = Bellman.error_handler(id: config.to_sym)
+          handler = Bellman.handler(id: config.to_sym)
           return handler if handler
 
           # rubocop:disable Lint/SuppressedException

data/lib/bellman/handlers/sentry.rb

@@ -47,9 +47,21 @@ module Bellman
       private
 
       def set_sentry_context(scope, objects)
+        # We are organizing the objects by class so that we can send the data
+        # on a per class basis. Note, we cannot just loop over the objects
+        # if we are using the class name as the key since this would mean that
+        # any time there are multiple objects of the same type, we'd only send
+        # the data for the last one as it would overwrite the data.
+        h = {}
         objects.each do |object|
+          key = object.class.name
+          h[key] = [] unless h.key?(key)
+          h[key] << object
+        end
+
+        h.each do |key, obj_array|
           scope.set_context(
-            object.class.name, { id: object.id }
+            key, { ids: obj_array.map(&:id) }
           )
         end
       end

data/lib/bellman/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Bellman
-  VERSION = '0.1.1'
+  VERSION = '0.1.2'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bellman
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Patrick R. Schmid