rookout 0.1.42 → 0.1.44

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1b7b251222795c20881099bbd6f64cb0e9c66c24f837403aecc29d5fc8737d22
-  data.tar.gz: 488333717b7f9a0ef507d5a621caa4a37a99bd87a3f966e63808ba773aeb83ef
+  metadata.gz: 0233712f50c0fe5e94c8288b73ee96a07da9bb1692cc553ac83692a10655c131
+  data.tar.gz: b273a8cc02ed25508400b58758c5706f6073a979c7d323316c9957dd4f186d30
 SHA512:
-  metadata.gz: fc9945c4fad24e0ebdee3bb83a1f71d353cef33bd14f9881007324ad2e82aba3292ba63cf54e54c14922547b1b68f6da58a1cdff99426dfd2a9bb85da78516cc
-  data.tar.gz: 1848074ffaa23f2632457e42dec7c04f4527cb4cad80af234e9bc7dc3c7c72f63c786ee7dfcae3e1e3bff44eb8665fcfabf5be7dcde2ead58eb3234d72909796
+  metadata.gz: afbacca6a3828ebb0e2bda728c072a9515fa6f3003945a9ab5bf972bde564a77dc950224d14bbc58b90aebdef3f505c90b89baf8d916beaca9c3bac3215fdb98
+  data.tar.gz: bf5c468000ed3dd7e2def2d628acab6540226ade8364a77036a12421ade338ed0106123f4c19cbc9e81c988e624bf50e9e84aa3dec090a17e68c41014390dc06

data/lib/rookout/augs/aug_factory.rb

@@ -107,7 +107,9 @@ module Rookout
 
         unless config.nil? || config.empty?
           rate_limit_split = config.split "/"
-          unless rate_limit_split.length == 2 && rate_limit_split[0].is_number? && rate_limit_split[1].is_number?
+          unless rate_limit_split.length == 2 && \
+                 Utils.is_number?(rate_limit_split[0]) && \
+                 Utils.is_number?(rate_limit_split[1])
             raise Exceptions::RookInvalidRateLimitConfiguration, config
           end
 

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

@@ -53,7 +53,7 @@ module Rookout
         def notify_warning error
           return if silence_log? error
 
-          Logger.instance.warning error.message
+          Logger.instance.warning error.message, error.exception
 
           # For easier testing
           return if @output.nil?

data/lib/rookout/com_ws/agent_com_ws.rb

@@ -105,7 +105,6 @@ module Rookout
 
       private
 
-      # rubocop:disable Style/StderrPuts
       def connection_thread
         backoff = Backoff.new
 
@@ -115,7 +114,8 @@ module Rookout
 
             if @print_on_initial_connection
               @print_on_initial_connection = false
-              $stderr.puts "[Rookout] Successfully connected to controller"
+              Utils.quiet_puts "[Rookout] Successfully connected to controller"
+              Utils.quiet_puts "[Rookout] Rook's ID is #{@agent_id}"
             end
             Logger.instance.debug "WebSocket connected successfully"
             Logger.instance.info "Finished initialization"
@@ -139,13 +139,11 @@ module Rookout
       rescue Exception => e
         Logger.instance.error "Unexpected error in connection_thread", e
       end
-      # rubocop:enable Style/StderrPuts
 
       def open_new_connection
         client = WebsocketClient.new @uri, @proxy, @token
         client.connect
 
-        Logger.instance.info "Registering agent with id #{@agent_id}"
         msg = Com::Rookout::NewAgentMessage.new agent_info: @info.pack
         client.send_frame wrap_in_envelope(msg)
 

data/lib/rookout/com_ws/envelope_wrapper.rb

@@ -29,7 +29,9 @@ class Variant2EnvelopeWrapper < EnvelopeWrapperBase
                                                              report_id: report_id
     @serializer = Rookout::Processor::NamespaceSerializer2.new
     @aug_report_message.arguments2 = @serializer.dump arguments, true
-    @serializer.string_cache.each { |key, value| @aug_report_message.strings_cache[key] = value }
+    @serializer.string_cache.each do |key, value|
+      @aug_report_message.strings_cache[key.encode "UTF-8", invalid: :replace, undef: :replace, replace: "?"] = value
+    end
     @envelope = wrap_in_envelope @aug_report_message
     @aug_report_message = nil
   end

data/lib/rookout/commit.rb

@@ -1,3 +1,3 @@
 module Rookout
-  COMMIT = "17ed82e31d57aaaecc1244c11f0e10e673069d7e".freeze
+  COMMIT = "426414a386441285302fbe4424892b2a98989af5".freeze
 end

data/lib/rookout/config.rb

@@ -6,8 +6,10 @@ module Rookout
     # Magic to allow for module variables to be easily accessible
     class << self
       attr_accessor :debug
+      attr_accessor :quiet
 
       Rookout::Config.debug = false
+      Rookout::Config.quiet = false
 
       attr_accessor :logger_filename
       attr_accessor :logger_log_to_stderr
@@ -112,7 +114,7 @@ module Rookout
           quota = configuration["RUBY_GLOBAL_RATE_LIMIT_QUOTA_MS"]
           window_size = configuration["RUBY_GLOBAL_RATE_LIMIT_WINDOW_SIZE_MS"]
 
-          if quota != "" && window_size != ""
+          if quota != "" && !quota.nil? && window_size != "" && !window_size.nil?
             global_rate_limit = "#{quota}/#{window_size}"
           end
         end

data/lib/rookout/exceptions.rb

@@ -241,7 +241,7 @@ module Rookout
       def initialize config
         super "Invalid rate limit configuration: #{config}",
               {
-                "rate_limit_config" => config
+                "config" => config
               }
       end
     end

data/lib/rookout/interface.rb

@@ -4,6 +4,7 @@ module Rookout
     include Singleton
 
     require_relative "config"
+    require_relative "utils"
     require_relative "exceptions"
     include Exceptions
 

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

@@ -91,23 +91,23 @@ module Rookout
         def dump log_object_errors
           dump_raw_object @obj, 0, @dump_config, log_object_errors
         end
-      end
 
-      private
+        private
 
-      def read_key_as_int key
-        key_int = key.to_i
-        return RubyObjectNamespace.new @obj[key_int] if key_int >= 0 && key_int < @obj.length
-        raise Exceptions::RookKeyNotFound, key
-      end
+        def read_key_as_int key
+          key_int = key.to_i
+          return RubyObjectNamespace.new @obj[key_int] if key_int >= 0 && key_int < @obj.length
+          raise Exceptions::RookKeyNotFound, key
+        end
 
-      def read_key_from_hash key
-        return RubyObjectNamespace.new @obj[key] if @obj.key? key
-        return RubyObjectNamespace.new @obj[key.to_sym] if @obj.key? key.to_s.to_sym
+        def read_key_from_hash key
+          return RubyObjectNamespace.new @obj[key] if @obj.key? key
+          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 }
+          @obj.each { |it, value| return RubyObjectNamespace.new value if it.to_s == key }
 
-        raise Exceptions::RookKeyNotFound, key
+          raise Exceptions::RookKeyNotFound, key
+        end
       end
     end
   end

data/lib/rookout/utils.rb

@@ -1,26 +1,34 @@
-module Utils
-  require "securerandom"
+module Rookout
+  module Utils
+    require "securerandom"
+    require_relative "config"
 
-  module_function
+    module_function
 
-  def uuid
-    SecureRandom.uuid.gsub(/-/, "")
-  end
+    def uuid
+      SecureRandom.uuid.gsub(/-/, "")
+    end
 
-  def milliseconds_to_nanoseconds milliseconds
-    nano = milliseconds * (10**6)
-    nano.to_i
-  end
+    def milliseconds_to_nanoseconds milliseconds
+      nano = milliseconds * (10**6)
+      nano.to_i
+    end
 
-  def time_to_nanoseconds time_obj
-    secs = time_obj.to_i
-    nsecs = time_obj.nsec
-    ((10**9) * secs) + nsecs
-  end
-end
+    def time_to_nanoseconds time_obj
+      secs = time_obj.to_i
+      nsecs = time_obj.nsec
+      ((10**9) * secs) + nsecs
+    end
+
+    def quiet_puts msg
+      # rubocop:disable Style/StderrPuts
+      return if Config.quiet
+      $stderr.puts msg
+      # rubocop:enable Style/StderrPuts
+    end
 
-class String
-  def is_number?
-    to_f.to_s == self || to_i.to_s == self
+    def is_number? string
+      true if Float(string) rescue false
+    end
   end
 end

data/lib/rookout/version.rb

@@ -1,3 +1,3 @@
 module Rookout
-  VERSION = "0.1.42".freeze
+  VERSION = "0.1.44".freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rookout
 version: !ruby/object:Gem::Version
-  version: 0.1.42
+  version: 0.1.44
 platform: ruby
 authors:
 - Liran Haimovitch
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-09-14 00:00:00.000000000 Z
+date: 2022-10-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: binding_of_caller