rookout 0.1.46 → 0.1.50

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b59aefa8a8063910a6a7ec3446c7f709a9781468e786993f2b7ae84b47c284cc
-  data.tar.gz: 91fc89ae75510ab8a81de5da86a3619c2f17d6a2920aa72d9f718436ac1b7056
+  metadata.gz: 72f208b3a47ef61e59b2123afe3e666a2f3efa8295ddec10950e221e5547770a
+  data.tar.gz: 216a478be4998de21db6b2df156123d6cde587a54369914d93ba2f3ef8f9b5a7
 SHA512:
-  metadata.gz: 0b9a998c2f766b42e115e103e92883807c813d9bdb583d6f9eaf148ead19de00d8fd71dae6a98a88dcfdbcd028634ff5de865410911b7ffcae92ff84bf224c20
-  data.tar.gz: b5ee7cf8ff1af74a6718d5abfd5d87e56ca7a521d470449abab7d07c525e32fa5f979e7b97553340dfd5a2b35618573ec10f4c2855cfc7ed24ccf5d2495abd07
+  metadata.gz: c9c2e1c91cba199a499fb360f519dcce9081f9c5693cf9919aaff369cd2389534900be7b580e03a3b9ef24e91394db0eec9d25e3968163de87a9390aec288392
+  data.tar.gz: 476c8d023ae8e04214693c3b929d0dae31bcf364cf23cc443bf3aab2c29bd05f08db4646a366cb54b4b5208d188df22024886bd35421849c73e331db720eb960

data/lib/rookout/augs/aug.rb

@@ -16,6 +16,7 @@ module Rookout
         @condition = condition
         @limits_manager = limits_manager
 
+        @executed = false
         @enabled = true
         @status = nil
         @log_cache = []
@@ -34,7 +35,9 @@ module Rookout
         namespace = create_namespaces frame, extracted
         return if @condition && !@condition.evaluate(namespace)
 
-        @limits_manager.with_limit do
+        should_skip_limiters = @condition.nil? && !@executed
+        @limits_manager.with_limit should_skip_limiters do
+          @executed = true
           report_id = Utils.uuid
           Logger.instance.info "Executing aug-\t#{id} (msg ID #{report_id})"
           @action.execute @id, report_id, namespace, output

data/lib/rookout/augs/aug_factory.rb

@@ -67,7 +67,7 @@ module Rookout
       def create_limits_manager configuration
         limiters = []
         if global_rate_limiter.nil?
-          rate_limit = parse_rate_limit configuration["rateLimit"], configuration["rateLimitModifier"], 200, 5000, 1
+          rate_limit = parse_rate_limit configuration["rateLimit"], configuration["rateLimitModifier"], 200, 5000
           unless rate_limit.nil?
             limiters.append AugRateLimiter.new(*rate_limit)
           end
@@ -82,7 +82,7 @@ module Rookout
         if @global_rate_limiter.nil? && Rookout::Config.global_rate_limit != ""
           begin
             rate_limit = parse_rate_limit Rookout::Config.global_rate_limit,
-                                          "0", 0, 0, Rookout::Config.global_rate_limit_multiplier
+                                          "0", 0, 0
             if rate_limit.nil?
               raise Exceptions::RookInvalidRateLimitConfiguration, Rookout::Config.global_rate_limit
             end
@@ -101,7 +101,7 @@ module Rookout
         @global_rate_limiter
       end
 
-      def parse_rate_limit config, modifier_config, default_quota, default_window, multiplier
+      def parse_rate_limit config, modifier_config, default_quota, default_window
         window_quota = default_quota
         window_size = default_window
 
@@ -129,7 +129,7 @@ module Rookout
           raise Exceptions::RookInvalidRateLimitConfiguration, config
         end
 
-        [window_quota_ns * multiplier, window_size_ns, rate_limit_modifier]
+        [window_quota_ns, window_size_ns, rate_limit_modifier]
       end
     end
   end

data/lib/rookout/augs/limits_manager.rb

@@ -5,13 +5,14 @@ module Rookout
         @limiters = limiters
       end
 
-      def with_limit start_time = nil
+      def with_limit skip_limiters, start_time = nil
         start_time ||= Time.now
         can_execute = true
         after_execute = []
 
         @limiters.each do |limiter|
-          if limiter.before_run start_time
+          limiter_passed = limiter.before_run start_time
+          if limiter_passed || skip_limiters
             after_execute.append -> { limiter.after_run start_time }
           else
             can_execute = false

data/lib/rookout/commit.rb

@@ -1,3 +1,3 @@
 module Rookout
-  COMMIT = "ec556ada7fc1b5196406db4ed39b386974440d91".freeze
+  COMMIT = "9f3ecb29fbadc0c8f66233c9c44be100a912e577".freeze
 end

data/lib/rookout/config.rb

@@ -81,13 +81,11 @@ module Rookout
       attr_accessor :global_rate_limit_quota
       attr_accessor :global_rate_limit_window_size
       attr_accessor :global_rate_limit
-      attr_accessor :global_rate_limit_multiplier
       attr_accessor :using_global_rate_limiter
 
       Rookout::Config.global_rate_limit_quota = ""
       Rookout::Config.global_rate_limit_window_size = ""
       Rookout::Config.global_rate_limit = ""
-      Rookout::Config.global_rate_limit_multiplier = 1
       Rookout::Config.using_global_rate_limiter = false
 
       def update_config configuration

data/lib/rookout/exceptions.rb

@@ -146,6 +146,13 @@ module Rookout
       end
     end
 
+    class RookInvalidPositionException < ToolException
+      def initialize filename, line
+        super "Invalid code position for TracePoint: #{filename}:#{line}",
+              { filename: filename, line: line }
+      end
+    end
+
     class RookObjectCannotBeSerialized < ToolException
       def initialize object, message
         super message, { "class" => object.class.to_s }

data/lib/rookout/services/tracer.rb

@@ -21,6 +21,9 @@ module Rookout
           begin
             trace_point.enable target: position.method, target_line: position.lineno
           rescue RuntimeError, ArgumentError => e
+            if e.message.include? "can not enable any hooks"
+              raise Exceptions::RookInvalidPositionException.new(aug.filename, position.lineno)
+            end
             raise Exceptions::RookSetTracepointFailed.new(position.lineno, e)
           end
 

data/lib/rookout/version.rb

@@ -1,3 +1,3 @@
 module Rookout
-  VERSION = "0.1.46".freeze
+  VERSION = "0.1.50".freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rookout
 version: !ruby/object:Gem::Version
-  version: 0.1.46
+  version: 0.1.50
 platform: ruby
 authors:
 - Liran Haimovitch
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-10-18 00:00:00.000000000 Z
+date: 2023-02-09 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.19.2
+        version: 3.21.7
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 3.19.2
+        version: 3.21.7
 - !ruby/object:Gem::Dependency
   name: google-style
   requirement: !ruby/object:Gem::Requirement
@@ -301,7 +301,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.7
+rubygems_version: 3.3.26
 signing_key:
 specification_version: 4
 summary: rookout is the Ruby SDK for the Rookout Debugging Platform