prefab-cloud-ruby 1.2.1 → 1.3.0

data/test/test_local_config_parser.rb

@@ -6,6 +6,11 @@ class TestLocalConfigParser < Minitest::Test
   FILE_NAME = 'example-config.yaml'
   DEFAULT_MATCH = 'default'
 
+  def setup
+    super
+    @mock_resolver = MockResolver.new
+  end
+
   def test_parse_int_config
     key = :sample_int
     parsed = Prefab::LocalConfigParser.parse(key, 123, {}, FILE_NAME)[key]
@@ -34,7 +39,7 @@ class TestLocalConfigParser < Minitest::Test
     assert_equal 1, config.rows[0].values.size
 
     value_row = config.rows[0].values[0]
-    assert_equal 'all-features', Prefab::ConfigValueUnwrapper.deepest_value(value_row.value, key, {}).unwrap
+    assert_equal 'all-features', Prefab::ConfigValueUnwrapper.deepest_value(value_row.value, key, {}, @mock_resolver).unwrap
   end
 
   def test_flag_in_user_key
@@ -53,13 +58,73 @@ class TestLocalConfigParser < Minitest::Test
     assert_equal 1, config.rows[0].values[0].criteria.size
 
     value_row = config.rows[0].values[0]
-    assert_equal true, Prefab::ConfigValueUnwrapper.deepest_value(value_row.value, key, {}).unwrap
+    assert_equal true, Prefab::ConfigValueUnwrapper.deepest_value(value_row.value, key, {}, @mock_resolver).unwrap
 
     assert_equal 'user.key', value_row.criteria[0].property_name
     assert_equal :PROP_IS_ONE_OF, value_row.criteria[0].operator
     assert_equal %w[abc123 xyz987], value_row.criteria[0].value_to_match.string_list.values
   end
 
+  def test_provided_values
+    with_env('LOOKUP_ENV', 'from env') do 
+      key = :test_provided
+      value = stringify_keys({type: 'provided', source: 'ENV_VAR', lookup: 'LOOKUP_ENV'})
+      parsed = Prefab::LocalConfigParser.parse(key, value, {}, FILE_NAME)[key]
+      config = parsed[:config]
+
+      assert_equal FILE_NAME, parsed[:source]
+      assert_equal 'LOOKUP_ENV', parsed[:match]
+      assert_equal :CONFIG, config.config_type
+      assert_equal key.to_s, config.key
+      assert_equal 1, config.rows.size
+      assert_equal 1, config.rows[0].values.size
+      
+      value_row = config.rows[0].values[0]            
+      provided = value_row.value.provided
+      assert_equal :ENV_VAR, provided.source
+      assert_equal 'LOOKUP_ENV', provided.lookup
+      assert_equal 'from env', Prefab::ConfigValueUnwrapper.deepest_value(value_row.value, config, {}, @mock_resolver).unwrap
+      reportable_value = Prefab::ConfigValueUnwrapper.deepest_value(value_row.value, config, {}, @mock_resolver).reportable_value
+      assert_equal 'from env', reportable_value
+    end
+  end
+
+  def test_confidential_provided_values
+    with_env('LOOKUP_ENV', 'from env') do
+      key = :test_provided
+      value = stringify_keys({type: 'provided', source: 'ENV_VAR', lookup: 'LOOKUP_ENV', confidential: true})
+      parsed = Prefab::LocalConfigParser.parse(key, value, {}, FILE_NAME)[key]
+      config = parsed[:config]
+
+      value_row = config.rows[0].values[0]
+      provided = value_row.value.provided
+      assert_equal :ENV_VAR, provided.source
+      assert_equal 'LOOKUP_ENV', provided.lookup
+      assert_equal 'from env', Prefab::ConfigValueUnwrapper.deepest_value(value_row.value, config, {}, @mock_resolver).unwrap
+      reportable_value = Prefab::ConfigValueUnwrapper.deepest_value(value_row.value, config, {}, @mock_resolver).reportable_value
+      assert reportable_value.start_with? Prefab::ConfigValueUnwrapper::CONFIDENTIAL_PREFIX
+    end
+  end
+
+  def test_confidential_values
+    key = :test_confidential
+    value = stringify_keys({value: 'a confidential string', confidential: true})
+    parsed = Prefab::LocalConfigParser.parse(key, value, {}, FILE_NAME)[key]
+    config = parsed[:config]
+
+    assert_equal FILE_NAME, parsed[:source]
+    assert_equal :CONFIG, config.config_type
+    assert_equal key.to_s, config.key
+    assert_equal 1, config.rows.size
+    assert_equal 1, config.rows[0].values.size
+
+    value_row = config.rows[0].values[0]
+    config_value = value_row.value
+    assert_equal 'a confidential string', Prefab::ConfigValueUnwrapper.deepest_value(config_value, key, {}, @mock_resolver).unwrap
+    reportable_value = Prefab::ConfigValueUnwrapper.deepest_value(config_value, key, {}, @mock_resolver).reportable_value
+    assert reportable_value.start_with? Prefab::ConfigValueUnwrapper::CONFIDENTIAL_PREFIX
+  end
+
   private
 
   def stringify_keys(hash)
@@ -73,4 +138,10 @@ class TestLocalConfigParser < Minitest::Test
     end
     result
   end
+
+  class MockResolver
+    def get(key)
+      raise "unexpected key"
+    end
+  end
 end

data/test/test_logger.rb

@@ -87,22 +87,22 @@ class TestLogger < Minitest::Test
 
   def test_log_internal
     prefab, io = captured_logger
-    prefab.log.log_internal(::Logger::WARN, 'test message', 'test.path')
+    prefab.log.log_internal(::Logger::WARN, 'test message', 'cloud.prefab.client.test.path')
     assert_logged io, 'WARN', "cloud.prefab.client.test.path", "test message"
   end
 
   def test_log_internal_unknown
     prefab, io = captured_logger
-    prefab.log.log_internal(::Logger::UNKNOWN, 'test message', 'test.path')
+    prefab.log.log_internal(::Logger::UNKNOWN, 'test message', 'cloud.prefab.client.test.path')
     assert_logged io, 'ANY', "cloud.prefab.client.test.path", "test message"
   end
 
   def test_log_internal_silencing
     prefab, io = captured_logger
     prefab.log.silence do
-      prefab.log.log_internal(::Logger::WARN, 'should not log', 'test.path')
+      prefab.log.log_internal(::Logger::WARN, 'should not log', 'cloud.prefab.client.test.path')
     end
-    prefab.log.log_internal(::Logger::WARN, 'should log', 'test.path')
+    prefab.log.log_internal(::Logger::WARN, 'should log', 'cloud.prefab.client.test.path')
     assert_logged io, 'WARN', "cloud.prefab.client.test.path", "should log"
     refute_logged io, 'should not log'
   end
@@ -457,7 +457,7 @@ class TestLogger < Minitest::Test
   def test_structured_internal_logging
     prefab, io = captured_logger
 
-    prefab.log.log_internal(::Logger::WARN, 'test', 'test.path', user: "michael")
+    prefab.log.log_internal(::Logger::WARN, 'test', 'cloud.prefab.client.test.path', user: "michael")
 
     assert_logged io, 'WARN', 'cloud.prefab.client.test.path', "test user=michael"
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: prefab-cloud-ruby
 version: !ruby/object:Gem::Version
-  version: 1.2.1
+  version: 1.3.0
 platform: ruby
 authors:
 - Jeff Dwyer
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-11-01 00:00:00.000000000 Z
+date: 2023-11-13 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: activesupport
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '4'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '4'
 - !ruby/object:Gem::Dependency
   name: benchmark-ips
   requirement: !ruby/object:Gem::Requirement
@@ -207,7 +221,9 @@ files:
 - lib/prefab/context_shape.rb
 - lib/prefab/context_shape_aggregator.rb
 - lib/prefab/criteria_evaluator.rb
+- lib/prefab/encryption.rb
 - lib/prefab/error.rb
+- lib/prefab/errors/env_var_parse_error.rb
 - lib/prefab/errors/initialization_timeout_error.rb
 - lib/prefab/errors/invalid_api_key_error.rb
 - lib/prefab/errors/missing_default_error.rb
@@ -220,6 +236,7 @@ files:
 - lib/prefab/internal_logger.rb
 - lib/prefab/local_config_parser.rb
 - lib/prefab/log_path_aggregator.rb
+- lib/prefab/log_subscribers/action_controller_subscriber.rb
 - lib/prefab/logger_client.rb
 - lib/prefab/murmer3.rb
 - lib/prefab/options.rb
@@ -228,6 +245,7 @@ files:
 - lib/prefab/rate_limit_cache.rb
 - lib/prefab/resolved_config_presenter.rb
 - lib/prefab/sse_logger.rb
+- lib/prefab/static_logger.rb
 - lib/prefab/time_helpers.rb
 - lib/prefab/weighted_value_resolver.rb
 - lib/prefab/yaml_config_parser.rb
@@ -241,6 +259,7 @@ files:
 - test/support/mock_base_client.rb
 - test/support/mock_config_client.rb
 - test/support/mock_config_loader.rb
+- test/test_action_controller.rb
 - test/test_client.rb
 - test/test_config_client.rb
 - test/test_config_loader.rb
@@ -251,6 +270,7 @@ files:
 - test/test_context_shape.rb
 - test/test_context_shape_aggregator.rb
 - test/test_criteria_evaluator.rb
+- test/test_encryption.rb
 - test/test_evaluation_summary_aggregator.rb
 - test/test_example_contexts_aggregator.rb
 - test/test_exponential_backoff.rb