sqreen 1.14.2-java → 1.15.0-java

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d64ca79b53ae2456a2c51b3939ff14c2d41b8dd82a7ce89860d82005cc75e1e5
-  data.tar.gz: fdee9e1364ff14e4167ae07bc9c1d878a073904a32d79acccbc1a041d96555c0
+  metadata.gz: ae1a29b376a222af3ccfc344c0c1010e5043f5f249113e917038af356d8ac24e
+  data.tar.gz: 3c5bad999caff0579ee484efe534e7b2b2b36fa0f68fac9f8900422c816b4ed3
 SHA512:
-  metadata.gz: 5a4b42b30b81fa25b154ccb5dea089dcd46ddb54c08fcb7e55946c8b45a03c74e4420bba338590b48582b3fea2084111c76b7ddffc4ff7fe862a0670e94ff5aa
-  data.tar.gz: 78e5b14939b1c743335b17fdaf9bc3d2d954c1806326edf4c88a2a58e5a170f87cf642bd8750caed162ec0090220b0d733f02cb38d3714403c46d8654cc249be
+  metadata.gz: 571a374c56db67eaed6775693c261467299a41e0f183e1d1847281d9998ded1611ca4cd6805372bab3bdfa42e1b0fb98d00dcc4433ee2aea89284a3d3cde9779
+  data.tar.gz: dabdb93c32dced05d2d7b56e89fa6e829b0e83cf4e07b98c3b12c15ba82ace39e64210c08e16a4f7f059c2d6cdeea42a54f4f29a0646db36cea098d611d661e6

data/lib/sqreen/js/mini_racer_adapter.rb

@@ -1,5 +1,4 @@
-require 'weakref'
-require 'sqreen/runner' # for Sqreen.on_forked_worker?
+require 'digest'
 
 module Sqreen
   module Js
@@ -8,11 +7,12 @@ module Sqreen
     class MiniRacerAdapter < JsServiceAdapter
       def initialize(vendored = false)
         @vendored = vendored
+        @pool = ContextPool.new
         self.class.static_init
       end
 
       def preprocess(_rule_name, code)
-        MiniRacerExecutableJs.new(code, @vendored)
+        MiniRacerExecutableJs.new(@pool, code, @vendored)
       end
 
       def variant_name
@@ -28,48 +28,74 @@ module Sqreen
 
     # Auxiliary classes
 
+    class ContextPool
+      def initialize
+        @mutex = Mutex.new
+        @total_ctxs = 0
+        @contexts = []
+      end
+
+      def with_context(&block)
+        isolate = get_context
+        begin
+          block[isolate]
+        ensure
+          give_back_context isolate
+        end
+      end
+
+      private
+
+      def get_context
+        @mutex.synchronize do
+          if @contexts.empty?
+            @total_ctxs += 1
+            Sqreen.log.debug "Creating new V8 context (#{@total_ctxs})"
+            SqreenContext.new
+          else
+            @contexts.pop
+          end
+        end
+      end
+
+      def give_back_context(isolate)
+        @mutex.synchronize { @contexts.push(isolate); }
+      end
+    end
+
     class MiniRacerExecutableJs < ExecutableJs
       @@ctx_defined = false
 
-      def initialize(source, vendored)
+      def initialize(pool, code, vendored)
+        @pool = pool
+        @code = code
+        @code_id = self.class.code_id(code)
+
         @module = vendored ? Sqreen::MiniRacer : MiniRacer
-        @source = source
-        @recycle_runtime_every = GC_MINI_RACER
-        @runtimes = []
-        @tl_key = "SQREEN_MINI_RACER_CONTEXT_#{object_id}".freeze
-        snapshot if Sqreen.on_forked_worker? # called to eagerly initialize snapshot
+
+        mod = vendored ? Sqreen::MiniRacer : MiniRacer
         unless @@ctx_defined
-          self.class.define_sqreen_context(@module)
+          self.class.define_sqreen_context(mod)
           @@ctx_defined = true
         end
       end
 
       def run_js_cb(cb_name, budget, arguments)
-        mini_racer_context = Thread.current[@tl_key]
-        dead_runtime = !mini_racer_context ||
-            !mini_racer_context[:r] || !mini_racer_context[:r].weakref_alive?
-        if !dead_runtime && mini_racer_context[:c] >= @recycle_runtime_every
-          dispose_runtime(mini_racer_context[:r])
-          dead_runtime = true
-        end
-        if dead_runtime
-          new_runtime = SqreenContext.new(:snapshot => snapshot)
-          push_runtime new_runtime
-          mini_racer_context = {
-              :c => 0,
-              :r => WeakCtx.new(new_runtime),
-          }
-          Thread.current[@tl_key] = mini_racer_context
+        @pool.with_context do |ctx|
+          ctx.add_code(@code_id, @code) unless ctx.has_code?(@code_id)
+
+          begin
+            json_args = "[#{arguments.map(&method(:fixup_bad_encoding)).map(&:to_json).join(',')}]"
+            ctx.eval_unsafe(
+                 "sqreen_data['#{@code_id}']['#{cb_name}'].apply(this, #{json_args})", nil, budget)
+          rescue @module::ScriptTerminatedError
+            nil
+          end
         end
+      end
 
-        mini_racer_context[:c] += 1
-        begin
-          json_args = "[#{arguments.map(&method(:fixup_bad_encoding)).map(&:to_json).join(',')}]"
-          mini_racer_context[:r].eval_unsafe(
-              "#{cb_name}.apply(this, #{json_args})", nil, budget)
-        rescue @module::ScriptTerminatedError
-          nil
-        end
+      def self.code_id(code)
+        Digest::MD5.base64digest(code)
       end
 
       private
@@ -91,29 +117,25 @@ module Sqreen
         end
       end
 
-      def snapshot
-        @snapshot ||= @module::Snapshot.new(@source)
-      end
-
-      def push_runtime(runtime)
-        @runtimes.delete_if do |th, runt, _thid|
-          del = th.nil? || !th.weakref_alive? || !th.alive?
-          runt.dispose if del
-          del
-        end
-        @runtimes.push [WeakRef.new(Thread.current), runtime, Thread.current.object_id]
-      end
-
-      def dispose_runtime(runtime)
-        @runtimes.delete_if { |_th, _runt, thid| thid == Thread.current.object_id }
-        runtime.dispose
-      end
-
       class << self
         def define_sqreen_context(modoole)
           # Context specialized for Sqreen usage
           Sqreen::Js.const_set 'SqreenContext', Class.new(modoole.const_get('Context'))
           SqreenContext.class_eval do
+            def has_code?(code_id)
+              return false unless @code_ids
+              @code_ids.include?(code_id)
+            end
+
+            def add_code(code_id, code)
+              @code_ids ||= Set.new
+              # if it fails, we don't try again
+              @code_ids << code_id
+
+              eval_unsafe code
+              transf_global_funcs code_id
+            end
+
             def eval_unsafe(str, filename = nil, timeoutv = nil)
               # Beware, timeout could be kept in the context
               # if perf cap is removed after having been activated
@@ -127,21 +149,29 @@ module Sqreen
                 super(str, filename)
               end
             end
-          end
-        end
-      end
-    end
 
+            private
+
+            def transf_global_funcs(code_id)
+              eval_unsafe <<EOD
+                if (typeof sqreen_data === 'undefined') {
+                  sqreen_data = {};
+                }
+
+                group = {};
+                Object.keys(this).forEach(name => {
+                  if (typeof this[name] === "function") {
+                    group[name] = this[name];
+                    this[name] = undefined;
+                  }
+                });
+                sqreen_data['#{code_id}'] = group
+                delete group;
+EOD
 
-    # Weak ref to a context
-    # enables us to skip a method missing call
-    class WeakCtx < WeakRef
-      def initialize(*args)
-        super(*args)
-      end
-
-      def eval_unsafe(str, filename, timeoutv)
-        __getobj__.eval_unsafe(str, filename, timeoutv)
+            end
+          end
+        end
       end
     end
   end

data/lib/sqreen/metrics/binning.rb

@@ -55,6 +55,7 @@ module Sqreen
       private
 
       def bin_no(x)
+        return 1 if x <= 0
         res = 2 + (@inv_log_base * Math.log(x) + @add_parcel).floor
         res < 1 ? 1 : res
       end

data/lib/sqreen/rules.rb

@@ -77,8 +77,8 @@ module Sqreen
       js = hash_rule[Attrs::CALLBACKS]
 
       if js && !Sqreen::Js::JsService.instance.online?
-        unless @@issue_nojs_warn
-          @@issue_nojs_warn = true
+        unless @issue_nojs_warn
+          @issue_nojs_warn = true
           Sqreen.log.warn('No JavaScript engine is available. ' \
               'JavaScript callbacks will be ignored')
         end

data/lib/sqreen/version.rb

@@ -1,5 +1,5 @@
 # Copyright (c) 2015 Sqreen. All Rights Reserved.
 # Please refer to our terms for more information: https://www.sqreen.io/terms.html
 module Sqreen
-  VERSION = '1.14.2'.freeze
+  VERSION = '1.15.0'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sqreen
 version: !ruby/object:Gem::Version
-  version: 1.14.2
+  version: 1.15.0
 platform: java
 authors:
 - Sqreen
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2018-10-02 00:00:00.000000000 Z
+date: 2018-10-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement