promoted-ruby-client 0.1.4 → 0.1.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 121200a7e00beda3ecf9643712df3aab9794f95719f9f8d65903b9484bf9857d
-  data.tar.gz: 901e657b2391b76d53d1b50d743469033f50f3679c1919d48c5644621e8c7866
+  metadata.gz: e9911fa22cb387f4258e9261729f5b17baa6412ae2d9ddd82cb943e35c02aef0
+  data.tar.gz: 29a8ae5517ecbdf317671280c31942781b13cf7ba8d6f3f09b9658e3101ee24a
 SHA512:
-  metadata.gz: 54fd3b49e89e049fe39c45afaa10171fc38cf24df7e402e46f922e22e0b12ed6f570f48b0f273b14a7c6a731b98c0d17493ddc4c4636b30582e8a115b0323b92
-  data.tar.gz: 195b628e077cbb154423aa8156fb16410582885f72ce292ea444c25abc1ca523656bdb65f5632088d593f4579a06f78cbfcd19a17da813095bdd144ad4b91ad9
+  metadata.gz: 82d1b9721132cc813ddc614292111a0003423c881d836a8dd471115a20b2a2c0b2f0cbdee4d7eb2c12cae993000623012dd21bf8844bd05a9803796f60a99d56
+  data.tar.gz: 91e3466ee73398157ab6b1e41778a89f760aa9741b7a2fcb8c0edb4ad382933ab13a4526145290d70990b4cfa8599c28ca7d0c18d968de50b2aaa81af4768f6c

data/Gemfile.lock

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

data/README.md

@@ -28,10 +28,10 @@ This client will suffice for building log requests. To send actually send traffi
 
 ```rb
 client = Promoted::Ruby::Client::PromotedClient.new({
-  :metrics_endpoint = "https://<get this from Promoted>",
-  :delivery_endpoint = "https://<get this from Promoted>",
-  :metrics_api_key = "<get this from Promoted>",
-  :delivery_api_key = "<get this from Promoted>"
+  :metrics_endpoint => "https://<get this from Promoted>",
+  :delivery_endpoint => "https://<get this from Promoted>",
+  :metrics_api_key => "<get this from Promoted>",
+  :delivery_api_key => "<get this from Promoted>"
 })
 ```
 
@@ -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

@@ -1,9 +1,8 @@
-
-## Deploy
+# Deploy
 
 1. Update version number.
 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.4.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

@@ -19,8 +19,16 @@ module Promoted
         class Error < StandardError; end
 
         attr_reader :perform_checks, :default_only_log, :delivery_timeout_millis, :metrics_timeout_millis, :should_apply_treatment_func,
-                    :default_request_headers, :http_client
+                    :default_request_headers, :http_client, :logger, :shadow_traffic_delivery_percent, :async_shadow_traffic
+                    
+        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
@@ -39,7 +47,8 @@ module Promoted
             @perform_checks = params[:perform_checks]
           end
 
-          @logger = params[:logger] # Example:  Logger.new(STDERR, :progname => "promotedai")
+          @logger               = params[:logger] # Example:  Logger.new(STDERR, :progname => "promotedai")
+          @request_logging_on   = params[:request_logging_on] || false
 
           @default_request_headers = params[:default_request_headers] || {}
           @metrics_api_key = params[:metrics_api_key] || ''
@@ -66,21 +75,36 @@ module Promoted
           @http_client = FaradayHTTPClient.new
           @validator = Promoted::Ruby::Client::Validator.new
 
-          # Thread pool to process delivery of shadow traffic. Will silently drop excess requests beyond the queue
-          # size, and silently eat errors on the background threads.
-          @pool = Concurrent::ThreadPoolExecutor.new(
-            min_threads: 0,
-            max_threads: 10,
-            max_queue: 100,
-            fallback_policy: :discard
-          )
+          @async_shadow_traffic = true
+          if params[:async_shadow_traffic] != nil
+            @async_shadow_traffic = params[:async_shadow_traffic] || false
+          end
+
+          @pool = nil
+          if @async_shadow_traffic
+            # Thread pool to process delivery of shadow traffic. Will silently drop excess requests beyond the queue
+            # size, and silently eat errors on the background threads.
+            @pool = Concurrent::ThreadPoolExecutor.new(
+              min_threads: 0,
+              max_threads: 10,
+              max_queue: 100,
+              fallback_policy: :discard
+            )
+          end
+
+          @enabled = true
+          if params[:enabled] != nil
+            @enabled = params[:enabled] || false
+          end
         end
         
         ##
         # Politely shut down a Promoted client.
         def close
-          @pool.shutdown
-          @pool.wait_for_termination
+          if @pool
+            @pool.shutdown
+            @pool.wait_for_termination
+          end
         end
 
         ##
@@ -88,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)
 
@@ -97,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
@@ -117,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] : nil)
+                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
@@ -156,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 = {
@@ -173,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
@@ -210,30 +242,47 @@ 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
+
           headers["x-api-key"] = api_key
           use_headers = @default_request_headers.merge headers
           
-          if send_async
+          if @request_logging_on && @logger
+            @logger.info("promotedai") {
+              "Sending #{payload.to_json} to #{endpoint}"
+            }
+          end
+
+          if send_async && @pool
             @pool.post do
-              @http_client.send(endpoint, timeout_millis, payload, use_headers)
+              start_time = Time.now.to_i
+              begin
+                resp = @http_client.send(endpoint, timeout_millis, payload, use_headers)
+              rescue Faraday::Error => err
+                @logger.warn("Deliver call failed with #{err}") if @logger
+                return
+              end
+              ellapsed_time = Time.now.to_i - start_time
+              @logger.info("Deliver call completed in #{ellapsed_time} ms") if @logger
             end
           else
             begin
-              @http_client.send(endpoint, timeout_millis, payload, use_headers)
+              resp = @http_client.send(endpoint, timeout_millis, payload, use_headers)
             rescue Faraday::Error => err
               raise EndpointError.new(err)
             end
           end
-        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
+          return resp
         end
 
         def should_send_as_shadow_traffic?
@@ -290,4 +339,5 @@ require "promoted/ruby/client/request_builder"
 require "promoted/ruby/client/sampler"
 require "promoted/ruby/client/util"
 require "promoted/ruby/client/validator"
-require 'securerandom'
+require 'securerandom'
+require 'time'

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

@@ -10,8 +10,8 @@ module Promoted
                 @conn = Faraday.new do |f|
                     f.request :json
                     f.request :retry, max: 3
-                    f.adapter :net_http
                     f.use Faraday::Response::RaiseError # raises on 4xx and 5xx responses
+                    f.adapter :net_http
                 end
             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]
@@ -48,10 +53,15 @@ module Promoted
           params = {
             user_info: user_info,
             timing: timing,
-            cohort_membership: @experiment,
-            client_info: @client_info.merge({ :client_type => Promoted::Ruby::Client::CLIENT_TYPE['PLATFORM_SERVER'] })
+            client_info: @client_info.merge({ :client_type => Promoted::Ruby::Client::CLIENT_TYPE['PLATFORM_SERVER'] }),
+            platform_id: @platform_id,
+            view_id: @view_id,
+            session_id: @session_id,
+            use_case: @use_case,
+            search_query: request[:search_query],
+            properties: request[:properties],
+            paging: request[:paging]
           }
-          params[:request] = request
           params[:insertion] = should_compact ? compact_delivery_insertions : full_insertion
 
           params.clean!
@@ -88,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
@@ -120,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
@@ -131,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.
@@ -142,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.4"
+      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.4
+  version: 0.1.8
 platform: ruby
 authors:
 - scottmcmaster
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-06-30 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