promoted-ruby-client 0.1.7 → 0.1.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 38925e54b884c3e36db632faa2481cd99621846848c21c80def989e1539a87d7
-  data.tar.gz: 6c0b3caedbef47b9ed6a5c5c1c9e58368031521c6f36181ae1a4938bd9f796fd
+  metadata.gz: e9911fa22cb387f4258e9261729f5b17baa6412ae2d9ddd82cb943e35c02aef0
+  data.tar.gz: 29a8ae5517ecbdf317671280c31942781b13cf7ba8d6f3f09b9658e3101ee24a
 SHA512:
-  metadata.gz: eae233ce5d572a6e581a57572292edf212205ab15764ff41db98116506c548b0746a94be236d44dda1bb43d758005a25bef8bbcfb8816295f4de9f30efb75481
-  data.tar.gz: 5085fa33d2e4d040d134a69cc8af872b4c209bda2ddee1be1f51f3d7c3b948ac8300152a790fbcdadaa5dbb75902b94727d1b20575398d563b5aa6757cfc3ea4
+  metadata.gz: 82d1b9721132cc813ddc614292111a0003423c881d836a8dd471115a20b2a2c0b2f0cbdee4d7eb2c12cae993000623012dd21bf8844bd05a9803796f60a99d56
+  data.tar.gz: 91e3466ee73398157ab6b1e41778a89f760aa9741b7a2fcb8c0edb4ad382933ab13a4526145290d70990b4cfa8599c28ca7d0c18d968de50b2aaa81af4768f6c

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    promoted-ruby-client (0.1.7)
+    promoted-ruby-client (0.1.8)
 
 GEM
   remote: https://rubygems.org/

data/README.md

@@ -88,7 +88,7 @@ Field Name | Type | Optional? | Description
 ---------- | ---- | --------- | -----------
 ```:user_info``` | UserInfo | Yes | The user info structure.
 ```:insertion_id``` | String | Yes | Generated by the SDK (*do not set*)
-```:request_id``` | String | Yes | Generated by the SDK (*do not set*)
+```:request_id``` | String | Yes | Generated by the SDK when needed (*do not set*)
 ```:content_id``` | String | No | Identifier for the content to be shown, must be set.
 ```:properties``` | Properties | Yes | Any additional custom properties to associate. For v1 integrations, it is fine not to fill in all the properties.
 
@@ -101,7 +101,7 @@ A request for content insertions.
 Field Name | Type | Optional? | Description
 ---------- | ---- | --------- | -----------
 ```:user_info``` | UserInfo | Yes | The user info structure.
-```:request_id``` | String | Yes | Generated by the SDK (*do not set*)
+```:request_id``` | String | Yes | Generated by the SDK when needed (*do not set*)
 ```:use_case``` | String | Yes | One of the use case values, i.e. 'FEED' (see [constants.rb](https://github.com/promotedai/promoted-ruby-client/blob/main/lib/promoted/ruby/client/constants.rb)).
 ```:properties``` | Properties | Yes | Any additional custom properties to associate.
 ```:paging``` | Paging | Yes | Paging parameters (see TODO)

data/dev.md

@@ -4,5 +4,5 @@
 2. Get credentials for deployment from 1password.
 3. Modify `promoted-ruby-client.gemspec`'s push block.
 4. Run `gem build promoted-ruby-client.gemspec` to generate `gem`.
-5. Run (using new output) `gem push promoted-ruby-client-0.1.7.gem`
+5. Run (using new output) `gem push promoted-ruby-client-0.1.8.gem`
 6. Update README with new version.

data/lib/promoted/ruby/client.rb

@@ -21,8 +21,14 @@ module Promoted
         attr_reader :perform_checks, :default_only_log, :delivery_timeout_millis, :metrics_timeout_millis, :should_apply_treatment_func,
                     :default_request_headers, :http_client, :logger, :shadow_traffic_delivery_percent, :async_shadow_traffic
                     
-        attr_accessor :request_logging_on
+        attr_accessor :request_logging_on, :enabled
         
+        ##
+        # Whether or not the client is currently enabled for execution.
+        def enabled?
+          @enabled
+        end
+
         ##            
         # A common compact method implementation.
         def self.copy_and_remove_properties
@@ -85,6 +91,11 @@ module Promoted
               fallback_policy: :discard
             )
           end
+
+          @enabled = true
+          if params[:enabled] != nil
+            @enabled = params[:enabled] || false
+          end
         end
         
         ##
@@ -101,6 +112,14 @@ module Promoted
         def deliver args, headers={}
           args = Promoted::Ruby::Client::Util.translate_args(args)
 
+          # Respect the enabled state
+          if !@enabled
+            return {
+              insertion: apply_paging(args[:full_insertion], args[:request][:paging])
+              # No log request returned when disabled
+            }
+          end
+          
           delivery_request_builder = RequestBuilder.new
           delivery_request_builder.set_request_params(args)
 
@@ -110,7 +129,7 @@ module Promoted
   
           response_insertions = []
           cohort_membership_to_log = nil
-          insertions_from_promoted = false
+          insertions_from_delivery = false
 
           only_log = delivery_request_builder.only_log != nil ? delivery_request_builder.only_log : @default_only_log
           deliver_err = false
@@ -130,22 +149,16 @@ module Promoted
                 @logger.error("Error calling delivery: " + err.message) if @logger
               end
               
-              insertions_from_promoted = (response != nil && !deliver_err);
+              insertions_from_delivery = (response != nil && !deliver_err);
               response_insertions = delivery_request_builder.fill_details_from_response(
                 response ? response[:insertion] : [])
             end
           end
   
           request_to_log = nil
-          if !insertions_from_promoted then
+          if !insertions_from_delivery then
             request_to_log = delivery_request_builder.request
-            size = delivery_request_builder.request.dig(:paging, :size)
-            response_insertions = size != nil ? delivery_request_builder.full_insertion[0..size] : delivery_request_builder.full_insertion
-          end
-
-          if request_to_log
-            request_to_log[:request_id] = SecureRandom.uuid if not request_to_log[:request_id]
-            add_missing_ids_on_insertions! request_to_log, response_insertions
+            response_insertions = apply_paging(delivery_request_builder.full_insertion, delivery_request_builder.request[:paging])
           end
 
           log_req = nil
@@ -169,8 +182,8 @@ module Promoted
             # On a successful delivery request, we don't log the insertions
             # or the request since they are logged on the server-side.
             log_req = log_request_builder.log_request_params(
-              include_insertions: !insertions_from_promoted, 
-              include_request: !insertions_from_promoted)
+              include_insertions: !insertions_from_delivery, 
+              include_request: !insertions_from_delivery)
           end
 
           client_response = {
@@ -186,6 +199,12 @@ module Promoted
         def prepare_for_logging args, headers={}
           args = Promoted::Ruby::Client::Util.translate_args(args)
 
+          if !@enabled
+            return {
+              insertion: args[:full_insertion]
+            }
+          end
+
           log_request_builder = RequestBuilder.new
 
           # Note: This method expects as JSON (string keys) but internally, RequestBuilder
@@ -223,6 +242,14 @@ module Promoted
 
         private
 
+        def apply_paging full_insertion, paging
+          size = nil
+          if paging
+            size = paging[:size]
+          end
+          return size != nil ? full_insertion[0..size - 1] : full_insertion
+        end
+
         def send_request payload, endpoint, timeout_millis, api_key, headers={}, send_async=false
           resp = nil
 
@@ -258,15 +285,6 @@ module Promoted
           return resp
         end
 
-
-        def add_missing_ids_on_insertions! request, insertions
-          insertions.each do |insertion|
-            insertion[:insertion_id] = SecureRandom.uuid if not insertion[:insertion_id]
-            insertion[:session_id] = request[:session_id] if request[:session_id]
-            insertion[:request_id] = request[:request_id] if request[:request_id]
-          end
-        end
-
         def should_send_as_shadow_traffic?
           @sampler.sample_random?(@shadow_traffic_delivery_percent)
         end

data/lib/promoted/ruby/client/id_generator.rb

@@ -0,0 +1,15 @@
+module Promoted
+    module Ruby
+      module Client
+        class IdGenerator
+            def initialize;end
+
+            def newID
+                SecureRandom.uuid
+            end
+        end
+    end
+  end
+end
+
+require 'securerandom'

data/lib/promoted/ruby/client/request_builder.rb

@@ -8,7 +8,13 @@ module Promoted
 
         attr_accessor :timing, :user_info, :platform_id
 
-        def initialize;end
+        def initialize args = {}
+          if args[:id_generator]
+            @id_generator = args[:id_generator]
+          else
+            @id_generator = IdGenerator.new
+          end
+        end
 
         # Populates request parameters from the given arguments, presumed to be a hash of symbols.
         def set_request_params args = {}
@@ -21,7 +27,6 @@ module Promoted
           @view_id                 = request[:view_id]
           @use_case                = Promoted::Ruby::Client::USE_CASES[request[:use_case]] || Promoted::Ruby::Client::USE_CASES['UNKNOWN_USE_CASE']
           @full_insertion          = args[:full_insertion]
-          @request_id              = SecureRandom.uuid
           @user_info               = request[:user_info] || { :user_id => nil, :log_user_id => nil}
           @timing                  = request[:timing] || { :client_log_timestamp => Time.now.to_i }
           @to_compact_metrics_insertion_func       = args[:to_compact_metrics_insertion_func]
@@ -50,7 +55,6 @@ module Promoted
             timing: timing,
             client_info: @client_info.merge({ :client_type => Promoted::Ruby::Client::CLIENT_TYPE['PLATFORM_SERVER'] }),
             platform_id: @platform_id,
-            request_id: @request_id,
             view_id: @view_id,
             session_id: @session_id,
             use_case: @use_case,
@@ -94,8 +98,16 @@ module Promoted
             cohort_membership: @experiment,
             client_info: @client_info
           }
+
+          request[:request_id] = @id_generator.newID if not request[:request_id]
+
+          # Log request allows for multiple requests but here we only send one.
           params[:request] = [request] if include_request
-          params[:insertion] = compact_metrics_insertions if include_insertions
+
+          if include_insertions
+            params[:insertion] = compact_metrics_insertions if include_insertions
+            add_missing_ids_on_insertions! request, params[:insertion]
+          end
           
           params.clean!
         end
@@ -126,7 +138,7 @@ module Promoted
             insertion_obj                = Hash[insertion_obj]
             insertion_obj[:user_info]    = user_info
             insertion_obj[:timing]       = timing
-            insertion_obj[:insertion_id] = SecureRandom.uuid # generate random UUID
+            insertion_obj[:insertion_id] = @id_generator.newID
             insertion_obj[:request_id]   = request_id
             insertion_obj[:position]     = offset + index
             insertion_obj                = @to_compact_metrics_insertion_func.call(insertion_obj) if @to_compact_metrics_insertion_func
@@ -137,6 +149,14 @@ module Promoted
 
         private
 
+        def add_missing_ids_on_insertions! request, insertions
+          insertions.each do |insertion|
+            insertion[:insertion_id] = @id_generator.newID if not insertion[:insertion_id]
+            insertion[:session_id] = request[:session_id] if request[:session_id]
+            insertion[:request_id] = request[:request_id] if request[:request_id]
+          end
+        end
+
         # A list of the response Insertions.  This client expects lists to be truncated
         # already to request.paging.size.  If not truncated, this client will truncate
         # the list.
@@ -148,6 +168,6 @@ module Promoted
   end
 end
 
-require 'securerandom'
 require "promoted/ruby/client/constants"
 require "promoted/ruby/client/extensions"
+require "promoted/ruby/client/id_generator"

data/lib/promoted/ruby/client/version.rb

@@ -1,7 +1,7 @@
 module Promoted
   module Ruby
     module Client
-      VERSION = "0.1.7"
+      VERSION = "0.1.8"
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: promoted-ruby-client
 version: !ruby/object:Gem::Version
-  version: 0.1.7
+  version: 0.1.8
 platform: ruby
 authors:
 - scottmcmaster
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-07-14 00:00:00.000000000 Z
+date: 2021-07-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -75,6 +75,7 @@ files:
 - lib/promoted/ruby/client/errors.rb
 - lib/promoted/ruby/client/extensions.rb
 - lib/promoted/ruby/client/faraday_http_client.rb
+- lib/promoted/ruby/client/id_generator.rb
 - lib/promoted/ruby/client/request_builder.rb
 - lib/promoted/ruby/client/sampler.rb
 - lib/promoted/ruby/client/util.rb