prefab-cloud-ruby 0.21.0 → 0.23.0

data/lib/prefab_pb.rb

@@ -181,6 +181,24 @@ Google::Protobuf::DescriptorPool.generated_pool.build do
       optional :sequence_name, :string, 3
       optional :size, :int64, 4
     end
+    add_message "prefab.Loggers" do
+      repeated :loggers, :message, 1, "prefab.Logger"
+      optional :start_at, :int64, 2
+      optional :end_at, :int64, 3
+      optional :instance_hash, :string, 4
+      proto3_optional :namespace, :string, 5
+    end
+    add_message "prefab.Logger" do
+      optional :logger_name, :string, 1
+      proto3_optional :traces, :int64, 2
+      proto3_optional :debugs, :int64, 3
+      proto3_optional :infos, :int64, 4
+      proto3_optional :warns, :int64, 5
+      proto3_optional :errors, :int64, 6
+      proto3_optional :fatals, :int64, 7
+    end
+    add_message "prefab.LoggerReportResponse" do
+    end
     add_enum "prefab.ConfigType" do
       value :NOT_SET_CONFIG_TYPE, 0
       value :CONFIG, 1
@@ -236,6 +254,9 @@ module Prefab
   CreationResponse = ::Google::Protobuf::DescriptorPool.generated_pool.lookup("prefab.CreationResponse").msgclass
   IdBlock = ::Google::Protobuf::DescriptorPool.generated_pool.lookup("prefab.IdBlock").msgclass
   IdBlockRequest = ::Google::Protobuf::DescriptorPool.generated_pool.lookup("prefab.IdBlockRequest").msgclass
+  Loggers = ::Google::Protobuf::DescriptorPool.generated_pool.lookup("prefab.Loggers").msgclass
+  Logger = ::Google::Protobuf::DescriptorPool.generated_pool.lookup("prefab.Logger").msgclass
+  LoggerReportResponse = ::Google::Protobuf::DescriptorPool.generated_pool.lookup("prefab.LoggerReportResponse").msgclass
   ConfigType = ::Google::Protobuf::DescriptorPool.generated_pool.lookup("prefab.ConfigType").enummodule
   LogLevel = ::Google::Protobuf::DescriptorPool.generated_pool.lookup("prefab.LogLevel").enummodule
   OnFailure = ::Google::Protobuf::DescriptorPool.generated_pool.lookup("prefab.OnFailure").enummodule

data/lib/prefab_services_pb.rb

@@ -61,6 +61,20 @@ module Prefab
       rpc :GetAll, ::Prefab::Identity, ::Prefab::ConfigEvaluations
     end
 
+    Stub = Service.rpc_stub_class
+  end
+  module LoggerReportingService
+    class Service
+
+      include ::GRPC::GenericService
+
+      self.marshal_class_method = :encode
+      self.unmarshal_class_method = :decode
+      self.service_name = 'prefab.LoggerReportingService'
+
+      rpc :Send, ::Prefab::Loggers, ::Prefab::LoggerReportResponse
+    end
+
     Stub = Service.rpc_stub_class
   end
 end

data/prefab-cloud-ruby.gemspec

@@ -2,16 +2,16 @@
 # DO NOT EDIT THIS FILE DIRECTLY
 # Instead, edit Juwelier::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: prefab-cloud-ruby 0.21.0 ruby lib
+# stub: prefab-cloud-ruby 0.23.0 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "prefab-cloud-ruby".freeze
-  s.version = "0.21.0"
+  s.version = "0.23.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0".freeze) if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib".freeze]
   s.authors = ["Jeff Dwyer".freeze]
-  s.date = "2023-02-27"
+  s.date = "2023-03-28"
   s.description = "RateLimits & Config as a service".freeze
   s.email = "jdwyer@prefab.cloud".freeze
   s.extra_rdoc_files = [
@@ -48,6 +48,7 @@ Gem::Specification.new do |s|
     "lib/prefab/feature_flag_client.rb",
     "lib/prefab/internal_logger.rb",
     "lib/prefab/local_config_parser.rb",
+    "lib/prefab/log_path_collector.rb",
     "lib/prefab/logger_client.rb",
     "lib/prefab/murmer3.rb",
     "lib/prefab/noop_cache.rb",
@@ -74,7 +75,9 @@ Gem::Specification.new do |s|
     "test/test_helper.rb",
     "test/test_integration.rb",
     "test/test_local_config_parser.rb",
+    "test/test_log_path_collector.rb",
     "test/test_logger.rb",
+    "test/test_options.rb",
     "test/test_weighted_value_resolver.rb"
   ]
   s.homepage = "http://github.com/prefab-cloud/prefab-cloud-ruby".freeze
@@ -93,6 +96,7 @@ Gem::Specification.new do |s|
     s.add_runtime_dependency(%q<google-protobuf>.freeze, [">= 0"])
     s.add_runtime_dependency(%q<grpc>.freeze, [">= 0"])
     s.add_runtime_dependency(%q<ld-eventsource>.freeze, [">= 0"])
+    s.add_runtime_dependency(%q<uuid>.freeze, [">= 0"])
     s.add_development_dependency(%q<benchmark-ips>.freeze, [">= 0"])
     s.add_development_dependency(%q<bundler>.freeze, [">= 0"])
     s.add_development_dependency(%q<grpc-tools>.freeze, [">= 0"])
@@ -106,6 +110,7 @@ Gem::Specification.new do |s|
     s.add_dependency(%q<google-protobuf>.freeze, [">= 0"])
     s.add_dependency(%q<grpc>.freeze, [">= 0"])
     s.add_dependency(%q<ld-eventsource>.freeze, [">= 0"])
+    s.add_dependency(%q<uuid>.freeze, [">= 0"])
     s.add_dependency(%q<benchmark-ips>.freeze, [">= 0"])
     s.add_dependency(%q<bundler>.freeze, [">= 0"])
     s.add_dependency(%q<grpc-tools>.freeze, [">= 0"])

data/test/test_client.rb

@@ -84,6 +84,30 @@ class TestClient < Minitest::Test
     assert certs.length > 1
   end
 
+  def test_initialization_with_an_options_object
+    options_hash = {
+      namespace: 'test-namespace',
+      prefab_datasources: Prefab::Options::DATASOURCES::LOCAL_ONLY
+    }
+
+    options = Prefab::Options.new(options_hash)
+
+    client = Prefab::Client.new(options)
+
+    assert_equal client.namespace, 'test-namespace'
+  end
+
+  def test_initialization_with_a_hash
+    options_hash = {
+      namespace: 'test-namespace',
+      prefab_datasources: Prefab::Options::DATASOURCES::LOCAL_ONLY
+    }
+
+    client = Prefab::Client.new(options_hash)
+
+    assert_equal client.namespace, 'test-namespace'
+  end
+
   private
 
   def new_client(overrides = {})

data/test/test_config_resolver.rb

@@ -374,8 +374,4 @@ class TestConfigResolver < Minitest::Test
     resolver.update
     resolver
   end
-
-  def string_list(values)
-    Prefab::ConfigValue.new(string_list: Prefab::StringList.new(values: values))
-  end
 end

data/test/test_helper.rb

@@ -43,7 +43,7 @@ class MockConfigClient
     @config_values = config_values
   end
 
-  def get(key, default = nil)
+  def get(key, default = nil, _, _)
     @config_values.fetch(key, default)
   end
 
@@ -93,3 +93,19 @@ def new_client(overrides = {})
 
   Prefab::Client.new(options)
 end
+
+def string_list(values)
+  Prefab::ConfigValue.new(string_list: Prefab::StringList.new(values: values))
+end
+
+def inject_config(client, config)
+  resolver = client.config_client.instance_variable_get('@config_resolver')
+  store = resolver.instance_variable_get('@local_store')
+
+  store[config.key] = { config: config }
+end
+
+def inject_project_env_id(client, project_env_id)
+  resolver = client.config_client.instance_variable_get('@config_resolver')
+  resolver.project_env_id = project_env_id
+end

data/test/test_log_path_collector.rb

@@ -0,0 +1,56 @@
+# frozen_string_literal: true
+
+require 'test_helper'
+require 'timecop'
+
+class TestLogPathCollector < Minitest::Test
+  def test_sync
+    Timecop.freeze do
+      client = new_client(namespace: 'this.is.a.namespace')
+
+      2.times { client.log.info('here is a message') }
+      3.times { client.log.error('here is a message') }
+
+      requests = []
+
+      client.define_singleton_method(:request) do |*params|
+        requests.push(params)
+      end
+
+      client.log_path_collector.send(:sync)
+
+      # let the flush thread run
+      sleep 0.01 while requests.length == 0
+
+      assert_equal requests, [[
+        Prefab::LoggerReportingService,
+        :send,
+        {
+          req_options: {},
+          params: Prefab::Loggers.new(
+            loggers: [Prefab::Logger.new(logger_name: 'test.test_log_path_collector.test_sync',
+                                         infos: 2, errors: 3)],
+            start_at: (Time.now.utc.to_f * 1000).to_i,
+            end_at: (Time.now.utc.to_f * 1000).to_i,
+            instance_hash: client.instance_hash,
+            namespace: 'this.is.a.namespace'
+          )
+        }
+      ]]
+    end
+  end
+
+  private
+
+  def new_client(overrides = {})
+    options = Prefab::Options.new(**{
+      prefab_config_override_dir: 'none',
+      prefab_config_classpath_dir: 'test',
+      prefab_envs: ['unit_tests'],
+      api_key: '123-development-yourapikey-SDK',
+      collect_sync_interval: 1000 # we'll trigger sync manually in our test
+    }.merge(overrides))
+
+    Prefab::Client.new(options)
+  end
+end

data/test/test_logger.rb

@@ -2,7 +2,22 @@
 
 require 'test_helper'
 
-class TestCLogger < Minitest::Test
+class TestLogger < Minitest::Test
+  TEST_ENV_ID = 2
+  DEFAULT_VALUE = 'FATAL'
+  DEFAULT_ENV_VALUE = 'INFO'
+  DESIRED_VALUE = 'DEBUG'
+  WRONG_ENV_VALUE = 'ERROR'
+  PROJECT_ENV_ID = 1
+
+  DEFAULT_ROW = Prefab::ConfigRow.new(
+    values: [
+      Prefab::ConditionalValue.new(
+        value: Prefab::ConfigValue.new(log_level: DEFAULT_VALUE)
+      )
+    ]
+  )
+
   def setup
     Prefab::LoggerClient.send(:public, :get_path)
     Prefab::LoggerClient.send(:public, :get_loc_path)
@@ -47,37 +62,37 @@ class TestCLogger < Minitest::Test
   def test_level_of
     with_env('PREFAB_LOG_CLIENT_BOOTSTRAP_LOG_LEVEL', 'info') do
       # env var overrides the default level
-      assert_equal Logger::INFO,
+      assert_equal ::Logger::INFO,
                    @logger.level_of('app.models.user'), 'PREFAB_LOG_CLIENT_BOOTSTRAP_LOG_LEVEL is info'
 
       @logger.set_config_client(MockConfigClient.new({}))
-      assert_equal Logger::WARN,
+      assert_equal ::Logger::WARN,
                    @logger.level_of('app.models.user'), 'default is warn'
 
       @logger.set_config_client(MockConfigClient.new('log-level.app' => :INFO))
-      assert_equal Logger::INFO,
+      assert_equal ::Logger::INFO,
                    @logger.level_of('app.models.user')
 
       @logger.set_config_client(MockConfigClient.new('log-level.app' => :DEBUG))
-      assert_equal Logger::DEBUG,
+      assert_equal ::Logger::DEBUG,
                    @logger.level_of('app.models.user')
 
       @logger.set_config_client(MockConfigClient.new('log-level.app' => :DEBUG,
                                                      'log-level.app.models' => :ERROR))
-      assert_equal Logger::ERROR,
+      assert_equal ::Logger::ERROR,
                    @logger.level_of('app.models.user'), 'test leveling'
     end
   end
 
   def test_log_internal
     logger, mock_logdev = mock_logger_expecting(/W, \[.*\]  WARN -- cloud.prefab.client.test.path: : test message/)
-    logger.log_internal('test message', 'test.path', '', Logger::WARN)
+    logger.log_internal('test message', 'test.path', '', ::Logger::WARN)
     mock_logdev.verify
   end
 
   def test_log_internal_unknown
     logger, mock_logdev = mock_logger_expecting(/A, \[.*\]   ANY -- cloud.prefab.client.test.path: : test message/)
-    logger.log_internal('test message', 'test.path', '', Logger::UNKNOWN)
+    logger.log_internal('test message', 'test.path', '', ::Logger::UNKNOWN)
     mock_logdev.verify
   end
 
@@ -85,30 +100,30 @@ class TestCLogger < Minitest::Test
     logger, mock_logdev = mock_logger_expecting(/W, \[.*\]  WARN -- cloud.prefab.client.test.path: : should log/,
                                                 calls: 2)
     logger.silence do
-      logger.log_internal('should not log', 'test.path', '', Logger::WARN)
+      logger.log_internal('should not log', 'test.path', '', ::Logger::WARN)
     end
-    logger.log_internal('should log', 'test.path', '', Logger::WARN)
+    logger.log_internal('should log', 'test.path', '', ::Logger::WARN)
     mock_logdev.verify
   end
 
   def test_log
     logger, mock_logdev = mock_logger_expecting(/W, \[.*\]  WARN -- test.path: : test message/)
-    logger.log('test message', 'test.path', '', Logger::WARN)
+    logger.log('test message', 'test.path', '', ::Logger::WARN)
     mock_logdev.verify
   end
 
   def test_log_unknown
     logger, mock_logdev = mock_logger_expecting(/A, \[.*\]   ANY -- test.path: : test message/)
-    logger.log('test message', 'test.path', '', Logger::UNKNOWN)
+    logger.log('test message', 'test.path', '', ::Logger::UNKNOWN)
     mock_logdev.verify
   end
 
   def test_log_silencing
     logger, mock_logdev = mock_logger_expecting(/W, \[.*\]  WARN -- test.path: : should log/, calls: 2)
     logger.silence do
-      logger.log('should not log', 'test.path', '', Logger::WARN)
+      logger.log('should not log', 'test.path', '', ::Logger::WARN)
     end
-    logger.log('should log', 'test.path', '', Logger::WARN)
+    logger.log('should log', 'test.path', '', ::Logger::WARN)
     mock_logdev.verify
   end
 
@@ -160,10 +175,226 @@ class TestCLogger < Minitest::Test
     assert_logged io, 'ERROR', 'MY_PROGNAME test.test_logger.test_logging_with_a_progname_and_no_message', 'MY_PROGNAME'
   end
 
+  def test_logging_with_criteria_on_top_level_key
+    prefix = 'my.own.prefix'
+
+    config = Prefab::Config.new(
+      key: 'log-level',
+      rows: [
+        DEFAULT_ROW,
+
+        # wrong env
+        Prefab::ConfigRow.new(
+          project_env_id: TEST_ENV_ID,
+          values: [
+            Prefab::ConditionalValue.new(
+              criteria: [
+                Prefab::Criterion.new(
+                  operator: Prefab::Criterion::CriterionOperator::PROP_IS_ONE_OF,
+                  value_to_match: string_list(['hotmail.com', 'gmail.com']),
+                  property_name: 'email_suffix'
+                )
+              ],
+              value: Prefab::ConfigValue.new(log_level: WRONG_ENV_VALUE)
+            )
+          ]
+        ),
+
+        # correct env
+        Prefab::ConfigRow.new(
+          project_env_id: PROJECT_ENV_ID,
+          values: [
+            Prefab::ConditionalValue.new(
+              criteria: [
+                Prefab::Criterion.new(
+                  operator: Prefab::Criterion::CriterionOperator::PROP_IS_ONE_OF,
+                  value_to_match: string_list(['hotmail.com', 'gmail.com']),
+                  property_name: 'email_suffix'
+                )
+              ],
+              value: Prefab::ConfigValue.new(log_level: DESIRED_VALUE)
+            ),
+            Prefab::ConditionalValue.new(
+              value: Prefab::ConfigValue.new(log_level: DEFAULT_ENV_VALUE)
+            )
+          ]
+        )
+      ]
+    )
+
+    prefab, io = captured_logger(log_prefix: prefix)
+
+    inject_config(prefab, config)
+    inject_project_env_id(prefab, PROJECT_ENV_ID)
+
+    # without any context, the level should be the default for the env (info)
+    prefab.with_log_context(nil, {}) do
+      prefab.log.debug 'Test debug'
+      refute_logged io, 'Test debug'
+
+      prefab.log.info 'Test info'
+      assert_logged io, 'INFO', "#{prefix}.test.test_logger.test_logging_with_criteria_on_top_level_key", 'Test info'
+
+      prefab.log.error 'Test error'
+      assert_logged io, 'ERROR', "#{prefix}.test.test_logger.test_logging_with_criteria_on_top_level_key", 'Test error'
+    end
+
+    reset_io(io)
+
+    # with the wrong context, the level should be the default for the env (info)
+    prefab.with_log_context('user:1234', email_suffix: 'yahoo.com') do
+      prefab.log.debug 'Test debug'
+      refute_logged io, 'Test debug'
+
+      prefab.log.info 'Test info'
+      assert_logged io, 'INFO', "#{prefix}.test.test_logger.test_logging_with_criteria_on_top_level_key", 'Test info'
+
+      prefab.log.error 'Test error'
+      assert_logged io, 'ERROR', "#{prefix}.test.test_logger.test_logging_with_criteria_on_top_level_key", 'Test error'
+    end
+
+    reset_io(io)
+
+    # with the correct context, the level should be the desired value (debug)
+    prefab.with_log_context('user:1234', email_suffix: 'hotmail.com') do
+      prefab.log.debug 'Test debug'
+      assert_logged io, 'DEBUG', "#{prefix}.test.test_logger.test_logging_with_criteria_on_top_level_key", 'Test debug'
+
+      prefab.log.info 'Test info'
+      assert_logged io, 'INFO', "#{prefix}.test.test_logger.test_logging_with_criteria_on_top_level_key", 'Test info'
+
+      prefab.log.error 'Test error'
+      assert_logged io, 'ERROR', "#{prefix}.test.test_logger.test_logging_with_criteria_on_top_level_key", 'Test error'
+    end
+  end
+
+  def test_logging_with_criteria_on_key_path
+    prefix = 'my.own.prefix'
+
+    config = Prefab::Config.new(
+      key: 'log-level.my.own.prefix.test.test_logger',
+      rows: [
+        DEFAULT_ROW,
+
+        # wrong env
+        Prefab::ConfigRow.new(
+          project_env_id: TEST_ENV_ID,
+          values: [
+            Prefab::ConditionalValue.new(
+              criteria: [
+                Prefab::Criterion.new(
+                  operator: Prefab::Criterion::CriterionOperator::PROP_IS_ONE_OF,
+                  value_to_match: string_list(['hotmail.com', 'gmail.com']),
+                  property_name: 'email_suffix'
+                )
+              ],
+              value: Prefab::ConfigValue.new(log_level: WRONG_ENV_VALUE)
+            )
+          ]
+        ),
+
+        # correct env
+        Prefab::ConfigRow.new(
+          project_env_id: PROJECT_ENV_ID,
+          values: [
+            Prefab::ConditionalValue.new(
+              criteria: [
+                Prefab::Criterion.new(
+                  operator: Prefab::Criterion::CriterionOperator::PROP_IS_ONE_OF,
+                  value_to_match: string_list(['hotmail.com', 'gmail.com']),
+                  property_name: 'email_suffix'
+                )
+              ],
+              value: Prefab::ConfigValue.new(log_level: DESIRED_VALUE)
+            ),
+
+            Prefab::ConditionalValue.new(
+              criteria: [
+                Prefab::Criterion.new(
+                  operator: Prefab::Criterion::CriterionOperator::LOOKUP_KEY_IN,
+                  value_to_match: string_list(%w[user:4567]),
+                  property_name: Prefab::CriteriaEvaluator::LOOKUP_KEY
+                )
+              ],
+              value: Prefab::ConfigValue.new(log_level: DESIRED_VALUE)
+            ),
+
+            Prefab::ConditionalValue.new(
+              value: Prefab::ConfigValue.new(log_level: DEFAULT_ENV_VALUE)
+            )
+          ]
+        )
+      ]
+    )
+
+    prefab, io = captured_logger(log_prefix: prefix)
+
+    inject_config(prefab, config)
+    inject_project_env_id(prefab, PROJECT_ENV_ID)
+
+    # without any context, the level should be the default for the env (info)
+    prefab.with_log_context(nil, {}) do
+      prefab.log.debug 'Test debug'
+      refute_logged io, 'Test debug'
+
+      prefab.log.info 'Test info'
+      assert_logged io, 'INFO', "#{prefix}.test.test_logger.test_logging_with_criteria_on_key_path", 'Test info'
+
+      prefab.log.error 'Test error'
+      assert_logged io, 'ERROR', "#{prefix}.test.test_logger.test_logging_with_criteria_on_key_path", 'Test error'
+    end
+
+    reset_io(io)
+
+    # with the wrong context, the level should be the default for the env (info)
+    prefab.with_log_context('user:1234', email_suffix: 'yahoo.com') do
+      prefab.log.debug 'Test debug'
+      refute_logged io, 'Test debug'
+
+      prefab.log.info 'Test info'
+      assert_logged io, 'INFO', "#{prefix}.test.test_logger.test_logging_with_criteria_on_key_path", 'Test info'
+
+      prefab.log.error 'Test error'
+      assert_logged io, 'ERROR', "#{prefix}.test.test_logger.test_logging_with_criteria_on_key_path", 'Test error'
+    end
+
+    reset_io(io)
+
+    # with the correct context, the level should be the desired value (debug)
+    prefab.with_log_context('user:1234', email_suffix: 'hotmail.com') do
+      prefab.log.debug 'Test debug'
+      assert_logged io, 'DEBUG', "#{prefix}.test.test_logger.test_logging_with_criteria_on_key_path", 'Test debug'
+
+      prefab.log.info 'Test info'
+      assert_logged io, 'INFO', "#{prefix}.test.test_logger.test_logging_with_criteria_on_key_path", 'Test info'
+
+      prefab.log.error 'Test error'
+      assert_logged io, 'ERROR', "#{prefix}.test.test_logger.test_logging_with_criteria_on_key_path", 'Test error'
+    end
+
+    reset_io(io)
+
+    # with the correct lookup key
+    prefab.with_log_context('user:4567', email_suffix: 'example.com') do
+      prefab.log.debug 'Test debug'
+      assert_logged io, 'DEBUG', "#{prefix}.test.test_logger.test_logging_with_criteria_on_key_path", 'Test debug'
+
+      prefab.log.info 'Test info'
+      assert_logged io, 'INFO', "#{prefix}.test.test_logger.test_logging_with_criteria_on_key_path", 'Test info'
+
+      prefab.log.error 'Test error'
+      assert_logged io, 'ERROR', "#{prefix}.test.test_logger.test_logging_with_criteria_on_key_path", 'Test error'
+    end
+  end
+
   private
 
   def assert_logged(logged_io, level, path, message)
-    assert_match(/#{level} \d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2} [-+]?\d+: #{path}: #{message}\n/, logged_io.string)
+    assert_match(/#{level}\s+\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2} [-+]?\d+:\s+#{path}: #{message}\n/, logged_io.string)
+  end
+
+  def refute_logged(logged_io, message)
+    refute_match(/#{message}/, logged_io.string)
   end
 
   def mock_logger_expecting(pattern, configs = {}, calls: 1)
@@ -192,4 +423,11 @@ class TestCLogger < Minitest::Test
 
     [prefab, io]
   end
+
+  def reset_io(io)
+    io.close
+    io.reopen
+
+    assert_equal '', io.string
+  end
 end

data/test/test_options.rb

@@ -0,0 +1,32 @@
+# frozen_string_literal: true
+
+require 'test_helper'
+
+class TestOptions < Minitest::Test
+  API_KEY = 'abcdefg'
+
+  def test_works_with_named_arguments
+    assert_equal API_KEY, Prefab::Options.new(api_key: API_KEY).api_key
+  end
+
+  def test_works_with_hash
+    assert_equal API_KEY, Prefab::Options.new({ api_key: API_KEY }).api_key
+  end
+
+  def test_collect_max_paths
+    assert_equal 1000, Prefab::Options.new.collect_max_paths
+    assert_equal 100, Prefab::Options.new(collect_max_paths: 100).collect_max_paths
+  end
+
+  def test_collect_max_paths_with_local_only
+    options = Prefab::Options.new(collect_max_paths: 100,
+                                  prefab_datasources: Prefab::Options::DATASOURCES::LOCAL_ONLY)
+    assert_equal 0, options.collect_max_paths
+  end
+
+  def test_collect_max_paths_with_collect_logs_false
+    options = Prefab::Options.new(collect_max_paths: 100,
+                                  collect_logs: false)
+    assert_equal 0, options.collect_max_paths
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: prefab-cloud-ruby
 version: !ruby/object:Gem::Version
-  version: 0.21.0
+  version: 0.23.0
 platform: ruby
 authors:
 - Jeff Dwyer
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-02-27 00:00:00.000000000 Z
+date: 2023-03-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: concurrent-ruby
@@ -100,6 +100,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: uuid
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: benchmark-ips
   requirement: !ruby/object:Gem::Requirement
@@ -221,6 +235,7 @@ files:
 - lib/prefab/feature_flag_client.rb
 - lib/prefab/internal_logger.rb
 - lib/prefab/local_config_parser.rb
+- lib/prefab/log_path_collector.rb
 - lib/prefab/logger_client.rb
 - lib/prefab/murmer3.rb
 - lib/prefab/noop_cache.rb
@@ -247,7 +262,9 @@ files:
 - test/test_helper.rb
 - test/test_integration.rb
 - test/test_local_config_parser.rb
+- test/test_log_path_collector.rb
 - test/test_logger.rb
+- test/test_options.rb
 - test/test_weighted_value_resolver.rb
 homepage: http://github.com/prefab-cloud/prefab-cloud-ruby
 licenses: