prefab-cloud-ruby 0.23.2 → 0.23.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6e93ba84e2ea630c852605f1bf814b5388dcaa8fd83025402d2dbc9afa8340cc
-  data.tar.gz: 8a751a537cbdf658015630b26e5cdc4d284bcc018c702712e60148e68890f0c0
+  metadata.gz: 74e19eed1d40cad8e3ea9b99a8755c90864a97b4a00e08d2df6a9fe5226f5ad0
+  data.tar.gz: 4ad94565a39139d01795007077128d8f997c05e6a8b785e019ca8b66308df5c5
 SHA512:
-  metadata.gz: f460a0d55982126a5fb1591d380936355ab175e93963dddf18aba9a5b6c2f599d36784afc6afe85aef01870d3069d3aab1cb52967a1a508f2d76aa6287b3599f
-  data.tar.gz: 1ce748c099c1414b98ca4476ec392e46068984196f4d6b261b6ee3acf696b86ae237ff63121a9a5050b500bd53fc957aadea7cec9d586cde75a3b8c4e543967c
+  metadata.gz: 7a10679fe049175ebaf01f12194f91bea1a98ce226bf0e0f3e87aea3ab299c888fb7cebfe67b942e77dc46367c9b05cc5acaf995870e1d6ee1f2b8aa744f2fa3
+  data.tar.gz: c6e506da285ae5f7913f81891e7f32a1b5365f1071bc90323f317aa2523fddb0bd41c2d8a53decf2afec000e234d535713e198f3ac8b4c98259c7904f03ce7ad

data/Gemfile

@@ -4,14 +4,12 @@ gem 'concurrent-ruby', '~> 1.0', '>= 1.0.5'
 gem 'faraday'
 gem 'googleapis-common-protos-types', platforms: :ruby
 gem 'google-protobuf', platforms: :ruby
-gem 'grpc', platforms: :ruby
 gem 'ld-eventsource'
 gem 'uuid'
 
 group :development do
   gem 'benchmark-ips'
   gem 'bundler'
-  gem 'grpc-tools', platforms: :ruby
   gem 'juwelier', '~> 2.4.9'
   gem 'rdoc'
   gem 'simplecov', '>= 0'

data/Gemfile.lock

@@ -33,10 +33,6 @@ GEM
     google-protobuf (3.22.2)
     googleapis-common-protos-types (1.5.0)
       google-protobuf (~> 3.14)
-    grpc (1.53.0)
-      google-protobuf (~> 3.21)
-      googleapis-common-protos-types (~> 1.0)
-    grpc-tools (1.43.1)
     hashie (3.6.0)
     highline (2.0.3)
     http (5.0.1)
@@ -124,8 +120,6 @@ DEPENDENCIES
   faraday
   google-protobuf
   googleapis-common-protos-types
-  grpc
-  grpc-tools
   juwelier (~> 2.4.9)
   ld-eventsource
   minitest

data/README.md

@@ -28,9 +28,7 @@ See full documentation https://docs.prefab.cloud/docs/ruby-sdk/ruby
 
 ## Important note about Forking and realtime updates
 
-Many ruby web servers fork. GRPC does not like to be forked. See some details on GRPC and forking: https://github.com/grpc/grpc/issues/7951#issuecomment-335998583
-
-If you're using Puma or Unicorn, you can do the following.
+Many ruby web servers fork. When the process is forked, the current realtime update stream is disconnected. If you're using Puma or Unicorn, do the following.
 
 ```ruby
 #config/initializers/prefab.rb

data/Rakefile

@@ -17,7 +17,7 @@ Juwelier::Tasks.new do |gem|
   gem.homepage = 'http://github.com/prefab-cloud/prefab-cloud-ruby'
   gem.license = 'MIT'
   gem.summary = %(Prefab Ruby Infrastructure)
-  gem.description = %(RateLimits & Config as a service)
+  gem.description = %(Feature Flags, Live Config, and Dynamic Log Levels as a service)
   gem.email = 'jdwyer@prefab.cloud'
   gem.authors = ['Jeff Dwyer']
 

data/VERSION

@@ -1 +1 @@
-0.23.2
+0.23.4

data/lib/prefab/client.rb

@@ -31,14 +31,8 @@ module Prefab
         @api_key = @options.api_key
         raise Prefab::Errors::InvalidApiKeyError, @api_key if @api_key.nil? || @api_key.empty? || api_key.count('-') < 1
 
-        @interceptor = Prefab::AuthInterceptor.new(@api_key)
         @prefab_api_url = @options.prefab_api_url
-        @prefab_grpc_url = @options.prefab_grpc_url
-        log_internal ::Logger::INFO,
-                     "Prefab Connecting to: #{@prefab_api_url} and #{@prefab_grpc_url} Secure: #{http_secure?}"
-        at_exit do
-          channel.destroy
-        end
+        log_internal ::Logger::INFO, "Prefab Connecting to: #{@prefab_api_url}"
       end
       # start config client
       config_client
@@ -54,19 +48,10 @@ module Prefab
       Thread.current[:prefab_log_properties] = {}
     end
 
-    def channel
-      credentials = http_secure? ? creds : :this_channel_is_insecure
-      @_channel ||= GRPC::Core::Channel.new(@prefab_grpc_url, nil, credentials)
-    end
-
     def config_client(timeout: 5.0)
       @config_client ||= Prefab::ConfigClient.new(self, timeout)
     end
 
-    def ratelimit_client(timeout: 5.0)
-      @ratelimit_client ||= Prefab::RateLimitClient.new(self, timeout)
-    end
-
     def feature_flag_client
       @feature_flag_client ||= Prefab::FeatureFlagClient.new(self)
     end
@@ -97,39 +82,6 @@ module Prefab
       log.log_internal msg, path, nil, level
     end
 
-    def request(service, method, req_options: {}, params: {})
-      # Future-proofing since we previously bumped into a conflict with a service with a `send` method
-      raise ArgumentError, 'Cannot call public_send on an grpc service in Ruby' if method.to_s == 'public_send'
-
-      opts = { timeout: 10 }.merge(req_options)
-
-      attempts = 0
-      start_time = Time.now
-
-      begin
-        attempts += 1
-
-        stub_for(service, opts[:timeout]).public_send(method, *params)
-      rescue StandardError => e
-        log_internal ::Logger::WARN, e
-
-        raise e if Time.now - start_time > opts[:timeout]
-
-        sleep_seconds = [BASE_SLEEP_SEC * (2**(attempts - 1)), MAX_SLEEP_SEC].min
-        sleep_seconds *= (0.5 * (1 + rand))
-        sleep_seconds = [BASE_SLEEP_SEC, sleep_seconds].max
-        log_internal ::Logger::INFO, "Sleep #{sleep_seconds} and Reset #{service} #{method}"
-        sleep sleep_seconds
-        reset!
-        retry
-      end
-    end
-
-    def reset!
-      @stubs.clear
-      @_channel = nil
-    end
-
     def enabled?(feature_name, lookup_key = nil, attributes = {})
       feature_flag_client.feature_is_on_for?(feature_name, lookup_key, attributes: attributes)
     end
@@ -142,6 +94,10 @@ module Prefab
       end
     end
 
+    def post(path, body)
+      Prefab::HttpConnection.new(@options.prefab_api_url, @api_key).post(path, body)
+    end
+
     private
 
     def is_ff?(key)
@@ -149,35 +105,5 @@ module Prefab
 
       raw && raw.allowable_values.any?
     end
-
-    def http_secure?
-      ENV['PREFAB_CLOUD_HTTP'] != 'true'
-    end
-
-    def stub_for(service, timeout)
-      @stubs["#{service}_#{timeout}"] ||= service::Stub.new(nil,
-                                                            nil,
-                                                            timeout: timeout,
-                                                            channel_override: channel,
-                                                            interceptors: [@interceptor])
-    end
-
-    def creds
-      GRPC::Core::ChannelCredentials.new(ssl_certs)
-    end
-
-    def ssl_certs
-      ssl_certs = ''
-      Dir["#{OpenSSL::X509::DEFAULT_CERT_DIR}/*.pem"].each do |cert|
-        ssl_certs += File.open(cert).read
-      end
-      if OpenSSL::X509::DEFAULT_CERT_FILE && File.exist?(OpenSSL::X509::DEFAULT_CERT_FILE)
-        ssl_certs += File.open(OpenSSL::X509::DEFAULT_CERT_FILE).read
-      end
-      ssl_certs
-    rescue StandardError => e
-      log.warn("Issue loading SSL certs #{e.message}")
-      ssl_certs
-    end
   end
 end

data/lib/prefab/config_client.rb

@@ -26,8 +26,6 @@ module Prefab
       @base_client.log_internal ::Logger::DEBUG, 'Initialize ConfigClient: AcquiredWriteLock'
       @initialized_future = Concurrent::Future.execute { @initialization_lock.acquire_read_lock }
 
-      @cancellable_interceptor = Prefab::CancellableInterceptor.new(@base_client)
-
       if @options.local_only?
         finish_init!(:local_only)
       else
@@ -58,11 +56,6 @@ module Prefab
       @config_resolver.update
     end
 
-    def reset
-      @base_client.reset!
-      @_stub = nil
-    end
-
     def to_s
       @config_resolver.to_s
     end
@@ -108,54 +101,28 @@ module Prefab
       @config_resolver.get(key, lookup_key, properties)
     end
 
-    def stub
-      @_stub = Prefab::ConfigService::Stub.new(nil,
-                                               nil,
-                                               channel_override: @base_client.channel,
-                                               interceptors: [@base_client.interceptor, @cancellable_interceptor])
-    end
-
     def load_checkpoint
       success = load_checkpoint_api_cdn
 
       return if success
 
-      @base_client.log_internal ::Logger::INFO, 'LoadCheckpoint: Fallback to GRPC API'
-
-      success = load_checkpoint_from_grpc_api
+      success = load_checkpoint_api
 
       return if success
 
       @base_client.log_internal ::Logger::WARN, 'No success loading checkpoints'
     end
 
-    def load_checkpoint_from_grpc_api
-      config_req = Prefab::ConfigServicePointer.new(start_at_id: @config_loader.highwater_mark)
-
-      resp = stub.get_all_config(config_req)
-      load_configs(resp, :remote_api_grpc)
-      true
-    rescue GRPC::Unauthenticated
-      @base_client.log_internal ::Logger::WARN, 'Unauthenticated'
-    rescue StandardError => e
-      @base_client.log_internal ::Logger::WARN, "Unexpected grpc_api problem loading checkpoint #{e}"
-      false
-    end
-
     def load_checkpoint_api_cdn
-      url = "#{@options.url_for_api_cdn}/api/v1/configs/0"
-      conn = if Faraday::VERSION[0].to_i >= 2
-               Faraday.new(url) do |conn|
-                 conn.request :authorization, :basic, AUTH_USER, @base_client.api_key
-               end
-             else
-               Faraday.new(url) do |conn|
-                 conn.request :basic_auth, AUTH_USER, @base_client.api_key
-               end
-             end
+      conn = Prefab::HttpConnection.new("#{@options.url_for_api_cdn}/api/v1/configs/0", @base_client.api_key)
       load_url(conn, :remote_cdn_api)
     end
 
+    def load_checkpoint_api
+      conn = Prefab::HttpConnection.new("#{@options.prefab_api_url}/api/v1/configs/0", @base_client.api_key)
+      load_url(conn, :remote_api)
+    end
+
     def load_url(conn, source)
       resp = conn.get('')
       if resp.status == 200

data/lib/prefab/exponential_backoff.rb

@@ -0,0 +1,16 @@
+module Prefab
+  class ExponentialBackoff
+    def initialize(max_delay:, initial_delay: 2, multiplier: 2)
+      @initial_delay = initial_delay
+      @max_delay = max_delay
+      @multiplier = multiplier
+      @delay = initial_delay
+    end
+
+    def call
+      delay = @delay
+      @delay = [@delay * @multiplier, @max_delay].min
+      delay
+    end
+  end
+end

data/lib/prefab/http_connection.rb

@@ -0,0 +1,37 @@
+# frozen_string_literal: true
+
+module Prefab
+  class HttpConnection
+    AUTH_USER = 'authuser'
+    PROTO_HEADERS = { 'Content-Type' => 'application/x-protobuf', 'Accept' => 'application/x-protobuf' }.freeze
+
+    def initialize(api_root, api_key)
+      @api_root = api_root
+      @api_key = api_key
+    end
+
+    def get(path)
+      connection.get(path)
+    end
+
+    def post(path, body)
+      connection(PROTO_HEADERS).post(path, body.to_proto)
+    end
+
+    def connection(headers = {})
+      if Faraday::VERSION[0].to_i >= 2
+        Faraday.new(@api_root) do |conn|
+          conn.request :authorization, :basic, AUTH_USER, @api_key
+
+          conn.headers.merge!(headers)
+        end
+      else
+        Faraday.new(@api_root) do |conn|
+          conn.request :basic_auth, AUTH_USER, @api_key
+
+          conn.headers.merge!(headers)
+        end
+      end
+    end
+  end
+end

data/lib/prefab/log_path_collector.rb

@@ -14,9 +14,13 @@ module Prefab
 
     def initialize(client:, max_paths:, sync_interval:)
       @max_paths = max_paths
-      @sync_interval = sync_interval
       @client = client
       @start_at = now
+      @sync_interval = if sync_interval.is_a?(Numeric)
+                         proc { sync_interval }
+                       else
+                         sync_interval || ExponentialBackoff.new(initial_delay: 8, max_delay: 60 * 10)
+                       end
 
       @pool = Concurrent::ThreadPoolExecutor.new(
         fallback_policy: :discard,
@@ -72,16 +76,16 @@ module Prefab
           namespace: @client.namespace
         )
 
-        @client.request Prefab::LoggerReportingService, :send, req_options: {}, params: loggers
+        @client.post('/api/v1/known-loggers', loggers)
       end
     end
 
     def start_periodic_sync
       Thread.new do
-        log_internal "Initialized log path collector instance_hash=#{@client.instance_hash} max_paths=#{@max_paths} sync_interval=#{@sync_interval}"
+        log_internal "Initialized log path collector instance_hash=#{@client.instance_hash} max_paths=#{@max_paths}"
 
         loop do
-          sleep @sync_interval
+          sleep @sync_interval.call
           sync
         end
       end

data/lib/prefab/options.rb

@@ -10,7 +10,6 @@ module Prefab
     attr_reader :shared_cache
     attr_reader :namespace
     attr_reader :prefab_api_url
-    attr_reader :prefab_grpc_url
     attr_reader :on_no_default
     attr_reader :initialization_timeout_sec
     attr_reader :on_init_failure
@@ -39,7 +38,6 @@ module Prefab
     end
 
     DEFAULT_MAX_PATHS = 1_000
-    DEFAULT_SYNC_INTERVAL = 60
 
     private def init(
       api_key: ENV['PREFAB_API_KEY'],
@@ -50,7 +48,6 @@ module Prefab
       log_formatter: DEFAULT_LOG_FORMATTER,
       log_prefix: nil,
       prefab_api_url: ENV['PREFAB_API_URL'] || 'https://api.prefab.cloud',
-      prefab_grpc_url: ENV['PREFAB_GRPC_URL'] || 'grpc.prefab.cloud:443',
       on_no_default: ON_NO_DEFAULT::RAISE, # options :raise, :warn_and_return_nil,
       initialization_timeout_sec: 10, # how long to wait before on_init_failure
       on_init_failure: ON_INITIALIZATION_FAILURE::RAISE, # options :unlock_and_continue, :lock_and_keep_trying, :raise
@@ -62,7 +59,7 @@ module Prefab
       prefab_envs: ENV['PREFAB_ENVS'].nil? ? [] : ENV['PREFAB_ENVS'].split(','),
       collect_logs: true,
       collect_max_paths: DEFAULT_MAX_PATHS,
-      collect_sync_interval: DEFAULT_SYNC_INTERVAL
+      collect_sync_interval: nil
     )
       @api_key = api_key
       @logdev = logdev
@@ -72,7 +69,6 @@ module Prefab
       @log_formatter = log_formatter
       @log_prefix = log_prefix
       @prefab_api_url = remove_trailing_slash(prefab_api_url)
-      @prefab_grpc_url = prefab_grpc_url
       @on_no_default = on_no_default
       @initialization_timeout_sec = initialization_timeout_sec
       @on_init_failure = on_init_failure

data/lib/prefab-cloud-ruby.rb

@@ -8,10 +8,10 @@ require 'openssl'
 require 'ld-eventsource'
 require 'prefab_pb'
 require 'prefab/error'
+require 'prefab/exponential_backoff'
 require 'prefab/errors/initialization_timeout_error'
 require 'prefab/errors/invalid_api_key_error'
 require 'prefab/errors/missing_default_error'
-require 'prefab_services_pb'
 require 'prefab/options'
 require 'prefab/internal_logger'
 require 'prefab/sse_logger'
@@ -22,13 +22,11 @@ require 'prefab/config_loader'
 require 'prefab/local_config_parser'
 require 'prefab/yaml_config_parser'
 require 'prefab/config_resolver'
+require 'prefab/http_connection'
 require 'prefab/client'
-require 'prefab/ratelimit_client'
 require 'prefab/config_client'
 require 'prefab/feature_flag_client'
 require 'prefab/logger_client'
-require 'prefab/auth_interceptor'
-require 'prefab/cancellable_interceptor'
 require 'prefab/noop_cache'
 require 'prefab/noop_stats'
 require 'prefab/murmer3'

data/prefab-cloud-ruby.gemspec

@@ -2,17 +2,17 @@
 # DO NOT EDIT THIS FILE DIRECTLY
 # Instead, edit Juwelier::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: prefab-cloud-ruby 0.23.2 ruby lib
+# stub: prefab-cloud-ruby 0.23.4 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "prefab-cloud-ruby".freeze
-  s.version = "0.23.2"
+  s.version = "0.23.4"
 
   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-04-04"
-  s.description = "RateLimits & Config as a service".freeze
+  s.date = "2023-04-12"
+  s.description = "Feature Flags, Live Config, and Dynamic Log Levels as a service".freeze
   s.email = "jdwyer@prefab.cloud".freeze
   s.extra_rdoc_files = [
     "LICENSE.txt",
@@ -33,8 +33,6 @@ Gem::Specification.new do |s|
     "VERSION",
     "compile_protos.sh",
     "lib/prefab-cloud-ruby.rb",
-    "lib/prefab/auth_interceptor.rb",
-    "lib/prefab/cancellable_interceptor.rb",
     "lib/prefab/client.rb",
     "lib/prefab/config_client.rb",
     "lib/prefab/config_loader.rb",
@@ -45,7 +43,9 @@ Gem::Specification.new do |s|
     "lib/prefab/errors/initialization_timeout_error.rb",
     "lib/prefab/errors/invalid_api_key_error.rb",
     "lib/prefab/errors/missing_default_error.rb",
+    "lib/prefab/exponential_backoff.rb",
     "lib/prefab/feature_flag_client.rb",
+    "lib/prefab/http_connection.rb",
     "lib/prefab/internal_logger.rb",
     "lib/prefab/local_config_parser.rb",
     "lib/prefab/log_path_collector.rb",
@@ -54,7 +54,6 @@ Gem::Specification.new do |s|
     "lib/prefab/noop_cache.rb",
     "lib/prefab/noop_stats.rb",
     "lib/prefab/options.rb",
-    "lib/prefab/ratelimit_client.rb",
     "lib/prefab/sse_logger.rb",
     "lib/prefab/weighted_value_resolver.rb",
     "lib/prefab/yaml_config_parser.rb",
@@ -71,6 +70,7 @@ Gem::Specification.new do |s|
     "test/test_config_resolver.rb",
     "test/test_config_value_unwrapper.rb",
     "test/test_criteria_evaluator.rb",
+    "test/test_exponential_backoff.rb",
     "test/test_feature_flag_client.rb",
     "test/test_helper.rb",
     "test/test_integration.rb",
@@ -94,12 +94,10 @@ Gem::Specification.new do |s|
     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<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"])
     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"])
@@ -108,12 +106,10 @@ Gem::Specification.new do |s|
     s.add_dependency(%q<faraday>.freeze, [">= 0"])
     s.add_dependency(%q<googleapis-common-protos-types>.freeze, [">= 0"])
     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"])
     s.add_dependency(%q<juwelier>.freeze, ["~> 2.4.9"])
     s.add_dependency(%q<rdoc>.freeze, [">= 0"])
     s.add_dependency(%q<simplecov>.freeze, [">= 0"])

data/test/integration_test.rb

@@ -91,8 +91,7 @@ class IntegrationTest
       prefab_envs: ['unit_tests'],
       prefab_datasources: Prefab::Options::DATASOURCES::ALL,
       api_key: ENV['PREFAB_INTEGRATION_TEST_API_KEY'],
-      prefab_api_url: 'https://api.staging-prefab.cloud',
-      prefab_grpc_url: 'grpc.staging-prefab.cloud:443'
+      prefab_api_url: 'https://api.staging-prefab.cloud'
     }.merge(@client_overrides))
   end
 end

data/test/test_client.rb

@@ -77,13 +77,6 @@ class TestClient < Minitest::Test
     assert_equal 'all-features', @client.get('flag_with_a_value')
   end
 
-  def test_ssl_certs
-    certs = @client.send(:ssl_certs).split('-----BEGIN CERTIFICATE-----')
-
-    # This is a smoke test to make sure multiple certs are loaded
-    assert certs.length > 1
-  end
-
   def test_initialization_with_an_options_object
     options_hash = {
       namespace: 'test-namespace',

data/test/test_exponential_backoff.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+require 'test_helper'
+
+class TestExponentialBackoff < Minitest::Test
+  def test_backoff
+    backoff = Prefab::ExponentialBackoff.new(max_delay: 120)
+
+    assert_equal 2, backoff.call
+    assert_equal 4, backoff.call
+    assert_equal 8, backoff.call
+    assert_equal 16, backoff.call
+    assert_equal 32, backoff.call
+    assert_equal 64, backoff.call
+    assert_equal 120, backoff.call
+    assert_equal 120, backoff.call
+  end
+end

data/test/test_log_path_collector.rb

@@ -4,6 +4,9 @@ require 'test_helper'
 require 'timecop'
 
 class TestLogPathCollector < Minitest::Test
+  MAX_WAIT = 2
+  SLEEP_TIME = 0.01
+
   def test_sync
     Timecop.freeze do
       client = new_client(namespace: 'this.is.a.namespace')
@@ -13,29 +16,32 @@ class TestLogPathCollector < Minitest::Test
 
       requests = []
 
-      client.define_singleton_method(:request) do |*params|
+      client.define_singleton_method(:post) do |*params|
         requests.push(params)
       end
 
       client.log_path_collector.send(:sync)
 
       # let the flush thread run
-      sleep 0.01 while requests.length == 0
+
+      wait_time = 0
+      while requests.length == 0
+        wait_time += SLEEP_TIME
+        sleep SLEEP_TIME
+
+        raise "Waited #{MAX_WAIT} seconds for the flush thread to run, but it never did" if wait_time > MAX_WAIT
+      end
 
       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'
-          )
-        }
+        '/api/v1/known-loggers',
+        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
@@ -47,8 +53,7 @@ class TestLogPathCollector < Minitest::Test
       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
+      api_key: '123-development-yourapikey-SDK'
     }.merge(overrides))
 
     Prefab::Client.new(options)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: prefab-cloud-ruby
 version: !ruby/object:Gem::Version
-  version: 0.23.2
+  version: 0.23.4
 platform: ruby
 authors:
 - Jeff Dwyer
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-04-04 00:00:00.000000000 Z
+date: 2023-04-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: concurrent-ruby
@@ -72,20 +72,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: grpc
-  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: ld-eventsource
   requirement: !ruby/object:Gem::Requirement
@@ -142,20 +128,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: grpc-tools
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: juwelier
   requirement: !ruby/object:Gem::Requirement
@@ -198,7 +170,7 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: RateLimits & Config as a service
+description: Feature Flags, Live Config, and Dynamic Log Levels as a service
 email: jdwyer@prefab.cloud
 executables: []
 extensions: []
@@ -220,8 +192,6 @@ files:
 - VERSION
 - compile_protos.sh
 - lib/prefab-cloud-ruby.rb
-- lib/prefab/auth_interceptor.rb
-- lib/prefab/cancellable_interceptor.rb
 - lib/prefab/client.rb
 - lib/prefab/config_client.rb
 - lib/prefab/config_loader.rb
@@ -232,7 +202,9 @@ files:
 - lib/prefab/errors/initialization_timeout_error.rb
 - lib/prefab/errors/invalid_api_key_error.rb
 - lib/prefab/errors/missing_default_error.rb
+- lib/prefab/exponential_backoff.rb
 - lib/prefab/feature_flag_client.rb
+- lib/prefab/http_connection.rb
 - lib/prefab/internal_logger.rb
 - lib/prefab/local_config_parser.rb
 - lib/prefab/log_path_collector.rb
@@ -241,7 +213,6 @@ files:
 - lib/prefab/noop_cache.rb
 - lib/prefab/noop_stats.rb
 - lib/prefab/options.rb
-- lib/prefab/ratelimit_client.rb
 - lib/prefab/sse_logger.rb
 - lib/prefab/weighted_value_resolver.rb
 - lib/prefab/yaml_config_parser.rb
@@ -258,6 +229,7 @@ files:
 - test/test_config_resolver.rb
 - test/test_config_value_unwrapper.rb
 - test/test_criteria_evaluator.rb
+- test/test_exponential_backoff.rb
 - test/test_feature_flag_client.rb
 - test/test_helper.rb
 - test/test_integration.rb

data/lib/prefab/auth_interceptor.rb

@@ -1,34 +0,0 @@
-# frozen_string_literal: true
-
-module Prefab
-  class AuthInterceptor < GRPC::ClientInterceptor
-    VERSION = File.exist?('VERSION') ? File.read('VERSION').chomp : ''
-    CLIENT = "prefab-cloud-ruby.#{VERSION}".freeze
-
-    def initialize(api_key)
-      @api_key = api_key
-    end
-
-    def request_response(request:, call:, method:, metadata:, &block)
-      shared(metadata, &block)
-    end
-
-    def client_streamer(requests:, call:, method:, metadata:, &block)
-      shared(metadata, &block)
-    end
-
-    def server_streamer(request:, call:, method:, metadata:, &block)
-      shared(metadata, &block)
-    end
-
-    def bidi_streamer(requests:, call:, method:, metadata:, &block)
-      shared(metadata, &block)
-    end
-
-    def shared(metadata)
-      metadata['auth'] = @api_key
-      metadata['client'] = CLIENT
-      yield
-    end
-  end
-end

data/lib/prefab/cancellable_interceptor.rb

@@ -1,49 +0,0 @@
-# frozen_string_literal: true
-
-module Prefab
-  class CancellableInterceptor < GRPC::ClientInterceptor
-    WAIT_SEC = 3
-
-    def initialize(base_client)
-      @base_client = base_client
-    end
-
-    def cancel
-      @call.instance_variable_get('@wrapped').instance_variable_get('@call').cancel
-      i = 0
-      while i < WAIT_SEC
-        if @call.instance_variable_get('@wrapped').cancelled?
-          @base_client.log_internal ::Logger::DEBUG, 'Cancelled streaming.'
-          return
-        else
-          @base_client.log_internal ::Logger::DEBUG, 'Unable to cancel streaming. Trying again'
-          @call.instance_variable_get('@wrapped').instance_variable_get('@call').cancel
-          i += 1
-          sleep(1)
-        end
-      end
-      @base_client.log_internal ::Logger::INFO, 'Unable to cancel streaming.'
-    end
-
-    def request_response(request:, call:, method:, metadata:, &block)
-      shared(call, &block)
-    end
-
-    def client_streamer(requests:, call:, method:, metadata:, &block)
-      shared(call, &block)
-    end
-
-    def server_streamer(request:, call:, method:, metadata:, &block)
-      shared(call, &block)
-    end
-
-    def bidi_streamer(requests:, call:, method:, metadata:, &block)
-      shared(call, &block)
-    end
-
-    def shared(call)
-      @call = call
-      yield
-    end
-  end
-end

data/lib/prefab/ratelimit_client.rb

@@ -1,78 +0,0 @@
-# frozen_string_literal: true
-
-module Prefab
-  class RateLimitClient
-    def initialize(base_client, timeout)
-      @timeout = timeout
-      @base_client = base_client
-    end
-
-    def pass?(group)
-      result = acquire([group], 1)
-      result.passed
-    end
-
-    def acquire(groups, acquire_amount, allow_partial_response: false, on_error: :log_and_pass)
-      expiry_cache_key = "prefab.ratelimit.expiry:#{groups.join('.')}"
-      expiry = @base_client.shared_cache.read(expiry_cache_key)
-      if !expiry.nil? && Integer(expiry) > Time.now.utc.to_f * 1000
-        @base_client.stats.increment('prefab.ratelimit.limitcheck.expirycache.hit', tags: [])
-        return Prefab::LimitResponse.new(passed: false, amount: 0)
-      end
-
-      req = Prefab::LimitRequest.new(
-        account_id: @base_client.account_id,
-        acquire_amount: acquire_amount,
-        groups: groups,
-        allow_partial_response: allow_partial_response
-      )
-
-      result = @base_client.request Prefab::RateLimitService, :limit_check, req_options: { timeout: @timeout },
-                                                                            params: req
-
-      reset = result.limit_reset_at
-      @base_client.shared_cache.write(expiry_cache_key, reset) unless reset < 1 # protobuf default int to 0
-
-      @base_client.stats.increment('prefab.ratelimit.limitcheck',
-                                   tags: ["policy_group:#{result.policy_group}", "pass:#{result.passed}"])
-
-      result
-    rescue StandardError => e
-      handle_error(e, on_error, groups)
-    end
-
-    def upsert(key, policy_name, limit, burst: nil, safety_level: nil)
-      burst = limit if burst.nil?
-      limit_definition = Prefab::LimitDefinition.new(
-        account_id: @base_client.account_id,
-        policy_name: Object.const_get("Prefab::LimitResponse::LimitPolicyNames::#{policy_name}"),
-        limit: limit,
-        burst: burst
-      )
-      limit_definition.safety_level = safety_level unless safety_level.nil?
-      config_value = Prefab::ConfigValue.new(limit_definition: limit_definition)
-      config_delta = Prefab::ConfigClient.value_to_delta(key, config_value)
-      upsert_req = Prefab::UpsertRequest.new(config_delta: config_delta)
-
-      @base_client.request Prefab::ConfigService, :upsert, req_options: { timeout: @timeout }, params: upsert_req
-    end
-
-    private
-
-    def handle_error(e, on_error, groups)
-      @base_client.stats.increment('prefab.ratelimit.error', tags: ['type:limit'])
-
-      message = "ratelimit for #{groups} error: #{e.message}"
-      case on_error
-      when :log_and_pass
-        @base_client.log.warn(message)
-        Prefab::LimitResponse.new(passed: true, amount: 0)
-      when :log_and_hit
-        @base_client.log.warn(message)
-        Prefab::LimitResponse.new(passed: false, amount: 0)
-      when :throw
-        raise e
-      end
-    end
-  end
-end