rookout 0.1.2 → 0.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 559c0c4b76d3815a418977ebe848a66f9640128be5a900286afedbf658a26e1e
-  data.tar.gz: dfeb5ba84b8b6665de0b0c1e5dec2fe687c3cb420485664f4d626bbb24213c7a
+  metadata.gz: ff23aa23db05a25f0bfa7f20ef0d7969d2b26d1a3480667f6165c95d4530ddc7
+  data.tar.gz: 4e5f6d7b5e9f34aa4cc9cd98c9b73152e7b25f73b49f02ee668d1f676ca8046e
 SHA512:
-  metadata.gz: d83bee904b048be0d9d4c221b2e5c797fa61a24a0dd43666b7211b8a5df917581aab351874cc819de11c853e08044454625d75d59594b197f8792d61e84f6d3c
-  data.tar.gz: e59f969f44d7f8311dbb96c7724be58900b45d294ce868742baf468856cbeba9219b14e67a32985619e98af895676df0fcc89ac6828fd89ac75b66d0d04f084b
+  metadata.gz: 658f60902cf42a6935b368d96ee466f3f0c2e341bf773961ced22d7a6bf706ccbc64ed91ea52c4bbf3df1a53bfec49f54c5285c99799494b1aca8fcee49c7593
+  data.tar.gz: 641ece8d6554713fa18e1b5c6999cb2b8783cd1e49114bee1ffc4843198cac6f4fca90f3f85a59781485c1d7017ee7047aaaf8c7ed94af0ec78d17d1668e9ba7

data/lib/rookout/augs/aug_factory.rb

@@ -7,6 +7,7 @@ module Rookout
     require_relative "actions/action_run_processor"
     require_relative "conditions/condition"
     require_relative "locations/location_file_line"
+    require_relative "locations/location_exception_handler"
     require_relative "aug_rate_limiter"
     require_relative "aug"
 
@@ -45,9 +46,15 @@ module Rookout
       def create_location configuration, aug
         name = configuration["name"]
         raise Exceptions.RookObjectNameMissing if name.nil?
-        raise Exceptions.RookUnsupportedLocation if name != "file_line"
 
-        Locations::LocationFileLine.new configuration, @output, aug
+        case name
+        when "file_line"
+          return Locations::LocationFileLine.new configuration, @output, aug
+        when "exception_handler"
+          return Locations::LocationExceptionHandler.new configuration, @output, aug
+        else
+          raise Exceptions.RookUnsupportedLocation if name != "file_line"
+        end
       end
 
       def create_rate_limit configuration

data/lib/rookout/augs/augs_manager.rb

@@ -2,6 +2,8 @@ module Rookout
   module Augs
     require_relative "../logger"
 
+    require_relative "../processor/rook_error"
+
     require_relative "aug_factory"
 
     class AugsManager
@@ -43,7 +45,7 @@ module Rookout
             return
           end
 
-          error = RookError.new e, message
+          error = Rookout::Processor::RookError.new e, message
           @output.send_rule_status aug_id, "Error", error
           return
         end

data/lib/rookout/augs/locations/location_exception_handler.rb

@@ -0,0 +1,22 @@
+module Rookout
+  module Augs
+    require_relative "../../logger"
+
+    require_relative "../../processor/rook_error"
+
+    module Locations
+      require_relative "location"
+
+      class LocationExceptionHandler < Location
+        NAME = "exception_handler".freeze
+
+        def initialize _arguments, output, aug
+          super output, aug
+        end
+
+        def add_aug _trigger_services
+        end
+      end
+    end
+  end
+end

data/lib/rookout/com_ws/output.rb

@@ -57,19 +57,13 @@ module Rookout
         return if @closing || !@agent_com
 
         @rule_status_update_bucket.if_available do
-          protobuf_error = nil
+          status = Com::Rookout::RuleStatusMessage.new agent_id: @agent_id,
+                                                       rule_id: rule_id,
+                                                       active: active
           if error
-            protobuf_error = Com::Rookout::Error.new message: error.message,
-                                                     type: error.type
-            protobuf_error.parameters.copy_from error.parameters
-            protobuf_error.exc.copy_from error.exc
-            protobuf_error.traceback.copy_from error.traceback
+            status.error = error.dumps
           end
 
-          status = Com::Rookout::RuleStatusMessage.new agent_id: @agent_id,
-                                                       rule_id: rule_id,
-                                                       active: active,
-                                                       error: protobuf_error
           @agent_com.add status
         end
       end

data/lib/rookout/interface.rb

@@ -84,7 +84,7 @@ module Rookout
       raise RookMissingToken if token.nil? && host == "wss://control.rookout.com"
       verify_token token if token
 
-      labels = options[:labels] || parse_labels(ENV["ROOKOUT_LABELS"])
+      labels = stringify_labels(options[:labels]) || parse_labels(ENV["ROOKOUT_LABELS"])
       validate_labels labels
 
       async_start = true? ENV["ROOKOUT_ASYNC_START"]
@@ -108,6 +108,18 @@ module Rookout
       default
     end
 
+    def stringify_labels labels
+      return nil unless labels
+
+      stringified_labels = {}
+
+      labels.each do |label_name, label_value|
+        stringified_labels[label_name.to_s] = label_value.to_s
+      end
+
+      stringified_labels
+    end
+
     def parse_labels raw_labels
       labels = {}
       return labels if raw_labels.nil?

data/lib/rookout/services/position.rb

@@ -98,7 +98,7 @@ module Rookout
           lineno = find_updated_line filename, location
           return if lineno == -1
 
-          PositionMarker.new location.lineno, iseq
+          PositionMarker.new lineno, iseq
         elsif suggested_match? location, filename
           warning = Exceptions::RookSourceFilePathSuggestion.new location.filename, filename
           location.notify_warning warning

data/lib/rookout/version.rb

@@ -1,3 +1,3 @@
 module Rookout
-  VERSION = "0.1.2".freeze
+  VERSION = "0.1.3".freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rookout
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
 platform: ruby
 authors:
 - Liran Haimovitch
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-12-07 00:00:00.000000000 Z
+date: 2020-12-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: binding_of_caller
@@ -197,6 +197,7 @@ files:
 - lib/rookout/augs/augs_manager.rb
 - lib/rookout/augs/conditions/condition.rb
 - lib/rookout/augs/locations/location.rb
+- lib/rookout/augs/locations/location_exception_handler.rb
 - lib/rookout/augs/locations/location_file_line.rb
 - lib/rookout/com_ws/agent_com_ws.rb
 - lib/rookout/com_ws/backoff.rb