sdk-reforge 1.9.0 → 1.9.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1c692e3a481a6340630232b56282b7e2dfa1e55102a03e910bb2522cc49a7660
-  data.tar.gz: e4485f45667650616b375e9b1d4aa76b03f3377744fb223d2807e4a30e99a7f4
+  metadata.gz: 9c9d01ab174a9a1de737bd5791203b24fe2b4e4edc029497233e704a3475f17f
+  data.tar.gz: 21a6556498ae0131164663d297890669b100ae3dccdaf6575a7e17b996ef988b
 SHA512:
-  metadata.gz: a378c5553acaf78760bdad194c3843f2d565a3cb8e66e4db3f462b5555748394674df9fa5a599eea144d533f6069dd3bc7a052bce3e604ede4193703e79fa719
-  data.tar.gz: 39b53e20a22e1c793bbb9bbe4dbde40c90dcea9ac0298be835fea3a22c222dec28e1cb12741fe8564952201de30db3e5700204430104df475c99c514cca3d611
+  metadata.gz: 2c6b7c7f36252e0e9434dc77a82ffc043c39f6085b405440a988ae2b8145ca127b90eaceca1b07d63ffa66d5cded48477d53fb9b2178a9afc5825966ec2d4724
+  data.tar.gz: f662153d281cce2fa52dbc4e8e7ad0c5864c896168a403900ca621f9e691a1e84c72c70186342b613bda714638c368c8e51074a5388f8d1a2ab542d47eff1755

data/.github/workflows/push_gem.yml

@@ -0,0 +1,46 @@
+---
+name: Push gem
+
+"on":
+  workflow_run:
+    workflows: ["Ruby"]
+    branches: [main]
+    types:
+      - completed
+
+jobs:
+  push:
+    runs-on: ubuntu-latest
+    if: ${{ github.event.workflow_run.conclusion == 'success' }}
+
+    permissions:
+      contents: write
+      id-token: write
+
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          persist-credentials: false
+          submodules: recursive
+
+      - name: Set up Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          bundler-cache: true
+          ruby-version: ruby
+
+      - name: Check if version already exists
+        id: version-check
+        run: |
+          VERSION=$(cat VERSION)
+          if gem list -r sdk-reforge | grep -q "sdk-reforge ($VERSION)"; then
+            echo "version-exists=true" >> $GITHUB_OUTPUT
+            echo "Version $VERSION already exists on RubyGems, skipping publish"
+          else
+            echo "version-exists=false" >> $GITHUB_OUTPUT
+            echo "Version $VERSION not found, proceeding with publish"
+          fi
+
+      - name: Release gem
+        if: steps.version-check.outputs.version-exists == 'false'
+        uses: rubygems/release-gem@v1

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 1.9.1 - 2025-10-01
+
+- Fix entrypoint
+
 
 ## 1.9.0 - 2025-08-23
 

data/Rakefile

@@ -56,8 +56,9 @@ end
 
 # Add release task for CI
 task :release do
-  sh 'gem build sdk-reforge.gemspec'
+  sh 'mkdir -p pkg'
   version = File.read('VERSION').strip
-  gem_file = "sdk-reforge-#{version}.gem"
+  gem_file = "pkg/sdk-reforge-#{version}.gem"
+  sh "gem build sdk-reforge.gemspec --output #{gem_file}"
   sh "gem push #{gem_file}"
 end

data/VERSION

@@ -1 +1 @@
-1.9.0
+1.9.1

data/dev/script_setup.rb

@@ -12,7 +12,7 @@ spec.require_paths.each do |path|
 end
 
 spec.require_paths.each do |path|
-  require "./lib/reforge-sdk"
+  require "./lib/sdk-reforge"
 end
 
 SemanticLogger.add_appender(io: $stdout)

data/lib/reforge/config_client.rb

@@ -140,6 +140,11 @@ module Reforge
     def load_url(conn, source)
       resp = conn.get('')
       if resp.status == 200
+        if resp.body.nil? || resp.body.empty?
+          LOG.warn "Checkpoint #{source} [#{conn.uri}] failed to load. Response body is empty"
+          return false
+        end
+
         configs = PrefabProto::Configs.decode(resp.body)
         load_configs(configs, source)
         cache_configs(configs)
@@ -218,7 +223,13 @@ module Reforge
       return false unless @options.use_local_cache
       File.open(cache_path) do |f|
         f.flock(File::LOCK_SH)
-        configs = PrefabProto::Configs.decode_json(f.read)
+        content = f.read
+        if content.nil? || content.empty?
+          LOG.warn "Failed to read cached configs at #{cache_path}. File is empty"
+          return false
+        end
+
+        configs = PrefabProto::Configs.decode_json(content)
         load_configs(configs, :cache)
 
         hours_old = ((Time.now - File.mtime(f)) / 60 / 60).round(2)
@@ -235,7 +246,13 @@ module Reforge
     def load_json_file(file)
       File.open(file) do |f|
         f.flock(File::LOCK_SH)
-        configs = PrefabProto::Configs.decode_json(f.read)
+        content = f.read
+        if content.nil? || content.empty?
+          LOG.warn "Failed to read datafile at #{file}. File is empty"
+          return false
+        end
+
+        configs = PrefabProto::Configs.decode_json(content)
         load_configs(configs, :datafile)
       end
     end

data/lib/reforge/{prefab.rb → reforge.rb}

@@ -117,4 +117,4 @@ module Reforge
       raise Reforge::Errors::UninitializedError.new(key)
     end
   end
-end
+end

data/lib/reforge/sse_config_client.rb

@@ -73,7 +73,20 @@ module Reforge
                       reconnect_time: @options.sse_default_reconnect_time,
                       logger: Reforge::InternalLogger.new(SSE::Client)) do |client|
         client.on_event do |event|
-          configs = PrefabProto::Configs.decode(Base64.decode64(event.data))
+          if event.data.nil? || event.data.empty?
+            @logger.error "SSE Streaming Error: Received empty data for url #{url}"
+            client.close
+            return
+          end
+
+          decoded_data = Base64.decode64(event.data)
+          if decoded_data.nil? || decoded_data.empty?
+            @logger.error "SSE Streaming Error: Decoded data is empty for url #{url}"
+            client.close
+            return
+          end
+
+          configs = PrefabProto::Configs.decode(decoded_data)
           load_configs.call(configs, event, :sse)
         end
 

data/lib/{reforge-sdk.rb → sdk-reforge.rb}

@@ -49,7 +49,7 @@ require 'reforge/client'
 require 'reforge/config_client_presenter'
 require 'reforge/config_client'
 require 'reforge/feature_flag_client'
-require 'reforge/prefab'
+require 'reforge/reforge'
 require 'reforge/murmer3'
 require 'reforge/javascript_stub'
 require 'reforge/semver'

data/sdk-reforge.gemspec

@@ -0,0 +1,149 @@
+# Generated by juwelier
+# DO NOT EDIT THIS FILE DIRECTLY
+# Instead, edit Juwelier::Tasks in Rakefile, and run 'rake gemspec'
+# -*- encoding: utf-8 -*-
+# stub: sdk-reforge 1.9.1 ruby lib
+
+Gem::Specification.new do |s|
+  s.name = "sdk-reforge".freeze
+  s.version = "1.9.1"
+
+  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 = "2025-10-01"
+  s.description = "Feature Flags, Live Config as a service".freeze
+  s.email = "jeff.dwyer@reforge.com.cloud".freeze
+  s.extra_rdoc_files = [
+    "CHANGELOG.md",
+    "LICENSE.txt",
+    "README.md"
+  ]
+  s.files = [
+    ".envrc.sample",
+    ".github/CODEOWNERS",
+    ".github/pull_request_template.md",
+    ".github/workflows/push_gem.yml",
+    ".github/workflows/ruby.yml",
+    ".gitmodules",
+    ".rubocop.yml",
+    ".tool-versions",
+    "CHANGELOG.md",
+    "CODEOWNERS",
+    "Gemfile",
+    "Gemfile.lock",
+    "LICENSE.txt",
+    "README.md",
+    "Rakefile",
+    "VERSION",
+    "compile_protos.sh",
+    "dev/allocation_stats",
+    "dev/benchmark",
+    "dev/console",
+    "dev/script_setup.rb",
+    "lib/prefab_pb.rb",
+    "lib/reforge/caching_http_connection.rb",
+    "lib/reforge/client.rb",
+    "lib/reforge/config_client.rb",
+    "lib/reforge/config_client_presenter.rb",
+    "lib/reforge/config_loader.rb",
+    "lib/reforge/config_resolver.rb",
+    "lib/reforge/config_value_unwrapper.rb",
+    "lib/reforge/config_value_wrapper.rb",
+    "lib/reforge/context.rb",
+    "lib/reforge/context_shape.rb",
+    "lib/reforge/context_shape_aggregator.rb",
+    "lib/reforge/criteria_evaluator.rb",
+    "lib/reforge/duration.rb",
+    "lib/reforge/encryption.rb",
+    "lib/reforge/error.rb",
+    "lib/reforge/errors/env_var_parse_error.rb",
+    "lib/reforge/errors/initialization_timeout_error.rb",
+    "lib/reforge/errors/invalid_sdk_key_error.rb",
+    "lib/reforge/errors/missing_default_error.rb",
+    "lib/reforge/errors/missing_env_var_error.rb",
+    "lib/reforge/errors/uninitialized_error.rb",
+    "lib/reforge/evaluation.rb",
+    "lib/reforge/evaluation_summary_aggregator.rb",
+    "lib/reforge/example_contexts_aggregator.rb",
+    "lib/reforge/exponential_backoff.rb",
+    "lib/reforge/feature_flag_client.rb",
+    "lib/reforge/fixed_size_hash.rb",
+    "lib/reforge/http_connection.rb",
+    "lib/reforge/internal_logger.rb",
+    "lib/reforge/javascript_stub.rb",
+    "lib/reforge/local_config_parser.rb",
+    "lib/reforge/murmer3.rb",
+    "lib/reforge/options.rb",
+    "lib/reforge/periodic_sync.rb",
+    "lib/reforge/rate_limit_cache.rb",
+    "lib/reforge/reforge.rb",
+    "lib/reforge/resolved_config_presenter.rb",
+    "lib/reforge/semver.rb",
+    "lib/reforge/sse_config_client.rb",
+    "lib/reforge/time_helpers.rb",
+    "lib/reforge/weighted_value_resolver.rb",
+    "lib/reforge/yaml_config_parser.rb",
+    "lib/sdk-reforge.rb",
+    "sdk-reforge.gemspec",
+    "test/fixtures/datafile.json",
+    "test/integration_test.rb",
+    "test/integration_test_helpers.rb",
+    "test/support/common_helpers.rb",
+    "test/support/mock_base_client.rb",
+    "test/support/mock_config_client.rb",
+    "test/support/mock_config_loader.rb",
+    "test/test_caching_http_connection.rb",
+    "test/test_client.rb",
+    "test/test_config_client.rb",
+    "test/test_config_loader.rb",
+    "test/test_config_resolver.rb",
+    "test/test_config_value_unwrapper.rb",
+    "test/test_config_value_wrapper.rb",
+    "test/test_context.rb",
+    "test/test_context_shape.rb",
+    "test/test_context_shape_aggregator.rb",
+    "test/test_criteria_evaluator.rb",
+    "test/test_duration.rb",
+    "test/test_encryption.rb",
+    "test/test_evaluation_summary_aggregator.rb",
+    "test/test_example_contexts_aggregator.rb",
+    "test/test_exponential_backoff.rb",
+    "test/test_feature_flag_client.rb",
+    "test/test_fixed_size_hash.rb",
+    "test/test_helper.rb",
+    "test/test_integration.rb",
+    "test/test_internal_logger.rb",
+    "test/test_javascript_stub.rb",
+    "test/test_local_config_parser.rb",
+    "test/test_logger_initialization.rb",
+    "test/test_options.rb",
+    "test/test_prefab.rb",
+    "test/test_rate_limit_cache.rb",
+    "test/test_semver.rb",
+    "test/test_sse_config_client.rb",
+    "test/test_weighted_value_resolver.rb"
+  ]
+  s.homepage = "http://github.com/ReforgeHQ/sdk-ruby".freeze
+  s.licenses = ["MIT".freeze]
+  s.rubygems_version = "3.4.19".freeze
+  s.summary = "Reforge Launch Ruby Infrastructure".freeze
+
+  s.specification_version = 4
+
+  s.add_runtime_dependency(%q<concurrent-ruby>.freeze, ["~> 1.0", ">= 1.0.5"])
+  s.add_runtime_dependency(%q<faraday>.freeze, [">= 0"])
+  s.add_runtime_dependency(%q<googleapis-common-protos-types>.freeze, [">= 0"])
+  s.add_runtime_dependency(%q<google-protobuf>.freeze, [">= 0"])
+  s.add_runtime_dependency(%q<ld-eventsource>.freeze, [">= 0"])
+  s.add_runtime_dependency(%q<uuid>.freeze, [">= 0"])
+  s.add_runtime_dependency(%q<activesupport>.freeze, [">= 4"])
+  s.add_runtime_dependency(%q<semantic_logger>.freeze, ["!= 4.16.0"])
+  s.add_development_dependency(%q<allocation_stats>.freeze, [">= 0"])
+  s.add_development_dependency(%q<benchmark-ips>.freeze, [">= 0"])
+  s.add_development_dependency(%q<bundler>.freeze, [">= 0"])
+  s.add_development_dependency(%q<juwelier>.freeze, ["~> 2.4.9"])
+  s.add_development_dependency(%q<rdoc>.freeze, [">= 0"])
+  s.add_development_dependency(%q<simplecov>.freeze, [">= 0"])
+end
+

data/test/test_config_client.rb

@@ -81,4 +81,71 @@ class TestConfigClient < Minitest::Test
     end
   end
 
+  def test_load_url_with_empty_body
+    options = Reforge::Options.new(
+      prefab_datasources: Reforge::Options::DATASOURCES::LOCAL_ONLY,
+      x_use_local_cache: true,
+      sdk_key: "123-ENV-KEY-SDK",)
+
+    config_client = Reforge::ConfigClient.new(MockBaseClient.new(options), 10)
+
+    # Mock connection with empty response body
+    mock_conn = Minitest::Mock.new
+    mock_resp = Minitest::Mock.new
+    mock_resp.expect(:status, 200)
+    mock_resp.expect(:body, '')
+    mock_resp.expect(:body, '')
+    mock_conn.expect(:get, mock_resp, [''])
+    mock_conn.expect(:uri, 'http://test.example.com')
+
+    result = config_client.send(:load_url, mock_conn, :test_source)
+
+    assert_equal false, result, 'Expected load_url to return false for empty body'
+    mock_conn.verify
+    mock_resp.verify
+
+    assert_logged [/Response body is empty/]
+  end
+
+  def test_load_cache_with_empty_file
+    options = Reforge::Options.new(
+      prefab_datasources: Reforge::Options::DATASOURCES::LOCAL_ONLY,
+      x_use_local_cache: true,
+      sdk_key: "123-ENV-KEY-SDK",)
+
+    config_client = Reforge::ConfigClient.new(MockBaseClient.new(options), 10)
+    cache_path = config_client.send(:cache_path)
+
+    # Create an empty cache file
+    FileUtils.mkdir_p(File.dirname(cache_path))
+    File.write(cache_path, '')
+
+    result = config_client.send(:load_cache)
+
+    assert_equal false, result, 'Expected load_cache to return false for empty file'
+    assert_logged [/File is empty/]
+  ensure
+    File.delete(cache_path) if File.exist?(cache_path)
+  end
+
+  def test_load_json_file_with_empty_file
+    options = Reforge::Options.new(
+      prefab_datasources: Reforge::Options::DATASOURCES::LOCAL_ONLY,
+      x_use_local_cache: true,
+      sdk_key: "123-ENV-KEY-SDK",)
+
+    config_client = Reforge::ConfigClient.new(MockBaseClient.new(options), 10)
+
+    # Create a temporary empty datafile
+    temp_file = File.join(Dir.tmpdir, 'test_empty_datafile.json')
+    File.write(temp_file, '')
+
+    result = config_client.send(:load_json_file, temp_file)
+
+    assert_equal false, result, 'Expected load_json_file to return false for empty file'
+    assert_logged [/File is empty/]
+  ensure
+    File.delete(temp_file) if File.exist?(temp_file)
+  end
+
 end

data/test/test_helper.rb

@@ -5,7 +5,7 @@ require 'minitest/focus'
 require 'minitest/reporters'
 Minitest::Reporters.use! unless ENV['RM_INFO']
 
-require 'reforge-sdk'
+require 'sdk-reforge'
 
 Dir.glob(File.join(File.dirname(__FILE__), 'support', '**', '*.rb')).each do |file|
   require file

data/test/test_sse_config_client.rb

@@ -208,4 +208,43 @@ class TestSSEConfigClient < Minitest::Test
       output << "data: CmYIu8fh4YaO0x4QZBo0bG9nLWxldmVsLmNsb3VkLnByZWZhYi5zZXJ2ZXIubG9nZ2luZy5FdmVudFByb2Nlc3NvciIfCAESG2phbWVzLmtlYmluZ2VyQHByZWZhYi5jbG91ZDgGSAkSDQhkELvH4eGGjtMeGGU=\n\n"
     end
   end
+
+  def test_empty_data_validation
+    # Unit test to verify that empty data is properly detected and handled
+    log_output = StringIO.new
+    logger = Logger.new(log_output)
+
+    # Test that empty event.data is detected
+    mock_event = OpenStruct.new(data: '')
+    mock_client = Minitest::Mock.new
+    mock_client.expect(:close, nil)
+
+    # Simulate the on_event handler logic
+    if mock_event.data.nil? || mock_event.data.empty?
+      logger.error "SSE Streaming Error: Received empty data for url http://test"
+      mock_client.close
+    end
+
+    log_lines = log_output.string.split("\n")
+    assert log_lines.any? { |line| line.include?('SSE Streaming Error') && line.include?('empty data') },
+           'Expected to have logged an error about empty data'
+    mock_client.verify
+
+    # Test that nil event.data is detected
+    log_output = StringIO.new
+    logger = Logger.new(log_output)
+    mock_event = OpenStruct.new(data: nil)
+    mock_client = Minitest::Mock.new
+    mock_client.expect(:close, nil)
+
+    if mock_event.data.nil? || mock_event.data.empty?
+      logger.error "SSE Streaming Error: Received empty data for url http://test"
+      mock_client.close
+    end
+
+    log_lines = log_output.string.split("\n")
+    assert log_lines.any? { |line| line.include?('SSE Streaming Error') && line.include?('empty data') },
+           'Expected to have logged an error about empty data for nil'
+    mock_client.verify
+  end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: sdk-reforge
 version: !ruby/object:Gem::Version
-  version: 1.9.0
+  version: 1.9.1
 platform: ruby
 authors:
 - Jeff Dwyer
 bindir: bin
 cert_chain: []
-date: 2025-09-23 00:00:00.000000000 Z
+date: 2025-10-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: concurrent-ruby
@@ -223,6 +223,7 @@ files:
 - ".envrc.sample"
 - ".github/CODEOWNERS"
 - ".github/pull_request_template.md"
+- ".github/workflows/push_gem.yml"
 - ".github/workflows/ruby.yml"
 - ".gitmodules"
 - ".rubocop.yml"
@@ -241,7 +242,6 @@ files:
 - dev/console
 - dev/script_setup.rb
 - lib/prefab_pb.rb
-- lib/reforge-sdk.rb
 - lib/reforge/caching_http_connection.rb
 - lib/reforge/client.rb
 - lib/reforge/config_client.rb
@@ -276,14 +276,16 @@ files:
 - lib/reforge/murmer3.rb
 - lib/reforge/options.rb
 - lib/reforge/periodic_sync.rb
-- lib/reforge/prefab.rb
 - lib/reforge/rate_limit_cache.rb
+- lib/reforge/reforge.rb
 - lib/reforge/resolved_config_presenter.rb
 - lib/reforge/semver.rb
 - lib/reforge/sse_config_client.rb
 - lib/reforge/time_helpers.rb
 - lib/reforge/weighted_value_resolver.rb
 - lib/reforge/yaml_config_parser.rb
+- lib/sdk-reforge.rb
+- sdk-reforge.gemspec
 - test/fixtures/datafile.json
 - test/integration_test.rb
 - test/integration_test_helpers.rb