rookout 0.1.27 → 0.1.30

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 29081116b19d38b4ff177eb1355605b69c735aa29cfe4b883d05099fe3556a5b
-  data.tar.gz: e39b5d4e540c790d194aea231ddf33a4d2743fa67ce2b42c2043c561e569873c
+  metadata.gz: f8bbc125cf28986d8ae700369a4d36c255e3912f0f9abb895ea10206d6090ca8
+  data.tar.gz: 3d20369b3dfe6141454e6a5a262defc0f6a0700d37d7ee6936911e4eaca5b510
 SHA512:
-  metadata.gz: 1cc4883a3543f2c6026e7c739662ab8a2ca418bad4b9335a285b7bc1b45e2c30ff63359bcd6ebd4b2a490a72dbacd9b8500b7656a343ac5ddf6d8217fbf63e2e
-  data.tar.gz: 0dffd5e418e7f81f467b1667df7370806458de425e9b2998991798226bbe7ee9367bf639b15c30b53244aa6c3dae0dc7616dd97cce70beef65ac2429cf877109
+  metadata.gz: 939d0c89c0683ca263ca0d0c7980c86537d681462d6e81fb447c0598a323fe7897f6342c3d51add8519e06295c33527d7488964b1a0bb448fcdb3b933c86fe24
+  data.tar.gz: f7d1228b2cdd81f6fe01a042b3aa38d89e7b6f2312c16cff32107dd6e0c40f951fe2aa67cb746b6f7adf076da245e422a1e3dbaa941063aa34af000db65c3aa5

data/lib/rookout/augs/aug_factory.rb

@@ -29,8 +29,9 @@ module Rookout
         max_aug_time = configuration["maxAugTime"] || Config.instrumentation_max_aug_time
 
         condition_configuration = configuration["conditional"]
-        raise Exceptions::RookAugInvalidKey.new("conditional", configuration) unless
-            condition_configuration.nil? || condition_configuration.is_a?(String)
+        unless condition_configuration.nil? || condition_configuration.is_a?(String)
+          raise Exceptions::RookAugInvalidKey.new("conditional", configuration)
+        end
         condition = condition_configuration.nil? ? nil : Conditions::Condition.new(condition_configuration)
 
         rate_limit = create_rate_limit configuration
@@ -47,15 +48,17 @@ module Rookout
 
       def create_location configuration, aug
         name = configuration["name"]
-        raise Exceptions::RookObjectNameMissing if name.nil?
+        raise Exceptions::RookObjectNameMissing, configuration if name.nil?
 
         case name
         when "file_line"
           return Locations::LocationFileLine.new configuration, @output, aug
         when "exception_handler"
           return Locations::LocationExceptionHandler.new configuration, @output, aug
+        when "log_handler"
+          raise Exceptions::RookUnsupportedLiveLogger
         else
-          raise Exceptions::RookUnsupportedLocation if name != "file_line"
+          raise Exceptions::RookUnsupportedLocation, configuration if name != "file_line"
         end
       end
 

data/lib/rookout/com_ws/agent_com_ws.rb

@@ -128,12 +128,14 @@ module Rookout
               @ready_event.set
             end
 
-            Logger.instance.info "Connection failed; reason = #{e.message}"
+            Logger.instance.warn "Connection failed; reason = #{e.message}"
           end
 
           backoff.after_disconnect
           Logger.instance.debug "Reconnecting"
         end
+      rescue Exception => e
+        Logger.instance.error "Unexpected error in connection_thread", e
       end
       # rubocop:enable Style/StderrPuts
 
@@ -157,7 +159,14 @@ module Rookout
       ].freeze
 
       def connection_pump client
-        on_outgoing_exit = proc { client.close }
+        on_outgoing_exit = proc {
+          begin
+            client.close
+          rescue Exception => e
+            Logger.instance.error "Unexpected error exiting outgoing thread", e
+          end
+        }
+
         @outgoing_thread = Thread.new { outgoing client, on_outgoing_exit }
         @outgoing_thread.name = "rookout-outgoing-thread"
 

data/lib/rookout/com_ws/websocket_client.rb

@@ -5,7 +5,6 @@ module Rookout
     require "uri"
 
     require_relative "../config"
-    require_relative "../logger"
     require_relative "../exceptions"
 
     class WebsocketClient
@@ -65,7 +64,7 @@ module Rookout
 
         begin
           @driver.close
-        rescue RuntimeError, Errno::EPIPE
+        rescue RuntimeError, Errno::EPIPE, OpenSSL::OpenSSLError
           # Protocol close may fail if the connection is already closed
           nil
         end

data/lib/rookout/commit.rb

@@ -1,3 +1,3 @@
 module Rookout
-  COMMIT = "25edb2f6d66577295d8ee34842e067d6ad046d12".freeze
+  COMMIT = "a753176d0b434bab7c3ab0467b5a544276d5a7f0".freeze
 end

data/lib/rookout/exceptions.rb

@@ -219,5 +219,11 @@ module Rookout
               }
       end
     end
+
+    class RookUnsupportedLiveLogger < ToolException
+      def initialize
+        super "Live Logger is not supported. Try using Rookout Live Debugger instead."
+      end
+    end
   end
 end

data/lib/rookout/interface.rb

@@ -43,7 +43,7 @@ module Rookout
         STDERR.puts "[Rookout] Failed to start Rookout: #{e.message}"
       rescue RookCommunicationException => e
         raise if throw_errors
-        STDERR.puts "[Rookout] " + e.message
+        Logger.instance.warn "[Rookout] " + e.message
       rescue Exception => e
         STDERR.puts e.full_message if Config.debug
         raise if throw_errors

data/lib/rookout/logger.rb

@@ -99,6 +99,7 @@ module Rookout
       end
     end
 
+    # rubocop:disable Style/RescueStandardError
     def log level, message, args
       level_no = LOG_LEVELS.index level
       if level_no.nil?
@@ -110,7 +111,11 @@ module Rookout
 
       record = LogRecord.new level, message, args
       @handlers.each { |handler| handler.call record }
+    rescue
+      # Log may never throw
+      nil
     end
+    # rubocop:enable Style/RescueStandardError
 
     def build_handlers
       if Config.logger_log_to_stderr

data/lib/rookout/processor/namespaces/ruby_object_namespace.rb

@@ -62,20 +62,20 @@ module Rookout
         end
 
         def read_key key
-          if @obj.is_a? Array
+          if @obj.is_a?(Array) || @obj.is_a?(String)
             key_int = key.to_i
             return RubyObjectNamespace.new @obj[key_int] if key_int >= 0 && key_int < @obj.length
             raise Exceptions::RookKeyNotFound, key
 
           elsif @obj.is_a? Hash
             return RubyObjectNamespace.new @obj[key] if @obj.key? key
-            return RubyObjectNamespace.new @obj[key.to_sym] if key.is_a?(String) && @obj.key?(key.to_sym)
+            return RubyObjectNamespace.new @obj[key.to_sym] if @obj.key? key.to_s.to_sym
 
             @obj.each { |it, value| return RubyObjectNamespace.new value if it.to_s == key }
 
             raise Exceptions::RookKeyNotFound, key
           else
-            raise Exceptions::RookInvalidObjectForAccess.new(obj.class.to_s, "ReadKey")
+            raise Exceptions::RookInvalidObjectForAccess.new(@obj.class.to_s, "ReadKey")
           end
         end
 

data/lib/rookout/services/position.rb

@@ -104,7 +104,7 @@ module Rookout
           PositionMarker.new lineno, iseq
         elsif suggested_match? location, filename
           warning = Exceptions::RookSourceFilePathSuggestion.new location.filename, filename
-          location.notify_warning warning
+          location.notify_warning Processor::RookError.new warning
           # Must return nil in order to skip this script
           nil
         end

data/lib/rookout/version.rb

@@ -1,3 +1,3 @@
 module Rookout
-  VERSION = "0.1.27".freeze
+  VERSION = "0.1.30".freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rookout
 version: !ruby/object:Gem::Version
-  version: 0.1.27
+  version: 0.1.30
 platform: ruby
 authors:
 - Liran Haimovitch
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-10-12 00:00:00.000000000 Z
+date: 2022-04-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: binding_of_caller
@@ -72,14 +72,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 3.17.3
+        version: 3.19.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 3.17.3
+        version: 3.19.2
 - !ruby/object:Gem::Dependency
   name: google-style
   requirement: !ruby/object:Gem::Requirement