ldclient-rb 3.0.0 → 3.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 5624d2c634cd8bc37ee54d2215e4150c754a2de7
-  data.tar.gz: da595a32a6ed8f80625318141639d1ddc26bed7f
+  metadata.gz: 2f3d50fce0854f03fc54889c382dff25d1bc7f07
+  data.tar.gz: 51dfad8e56e09fa7da457d55863223b3a64cd9f6
 SHA512:
-  metadata.gz: 79f7bff1fd78fdc74370ae3604a13da553cdc9052d4a3c294470c51ca7f4969c93b8f9a317bc906f5b90f0d7d2d997e7a64738e7f209a0267ede8eedfb74b952
-  data.tar.gz: f3288d3da6869ee47c2785d34d190d63391ba0d0df59cb3b135c939589f52e6cdc9645b7cd1503d51822214888b65ba0221e2d4341513633ee9ef03caa860fc9
+  metadata.gz: d3e036b4cf2ff3c032e3260be2fa73bb22c4ef0dc79467a8c877713d388b139ef4f76f3c5b666fdc0acf8455f909ef8b38b729d2950edb309fa8ee5d7db6ea30
+  data.tar.gz: 5d16aa332570cd295243e4ba507bfc99cbf8f30e12bca72d96767050d9ebfc3a16545bc118c0e8a36535dce447c81228913f536661d7f20c49be6aa270180dd0

data/CHANGELOG.md

@@ -2,6 +2,11 @@
 
 All notable changes to the LaunchDarkly Ruby SDK will be documented in this file. This project adheres to [Semantic Versioning](http://semver.org).
 
+## [3.0.1] - 2018-02-26
+### Fixed
+- Fixed a bug that could prevent very large feature flags from being updated in streaming mode.
+
+
 ## [3.0.0] - 2018-02-22
 ### Added
 - Support for a new LaunchDarkly feature: reusable user segments.

data/lib/ldclient-rb/requestor.rb

@@ -18,18 +18,10 @@ module LaunchDarkly
       end
     end
 
-    def request_all_flags()
-      make_request("/sdk/latest-flags")
-    end
-
     def request_flag(key)
       make_request("/sdk/latest-flags/" + key)
     end
 
-    def request_all_segments()
-      make_request("/sdk/latest-segments")
-    end
-
     def request_segment(key)
       make_request("/sdk/latest-segments/" + key)
     end

data/lib/ldclient-rb/stream.rb

@@ -110,13 +110,13 @@ module LaunchDarkly
         @initialized.make_true
         @config.logger.info("[LDClient] Stream initialized (via indirect message)")
       elsif method == INDIRECT_PATCH
-        key = feature_key_for_path(message.data)
+        key = key_for_path(FEATURES, message.data)
         if key
           @feature_store.upsert(FEATURES, @requestor.request_flag(key))
         else
-          key = segment_key_for_path(message.data)
+          key = key_for_path(SEGMENTS, message.data)
           if key
-            @feature_store.upsert(SEGMENTS, key, @requestor.request_segment(key))
+            @feature_store.upsert(SEGMENTS, @requestor.request_segment(key))
           end
         end
       else

data/lib/ldclient-rb/version.rb

@@ -1,3 +1,3 @@
 module LaunchDarkly
-  VERSION = "3.0.0"
+  VERSION = "3.0.1"
 end

data/spec/requestor_spec.rb

@@ -19,12 +19,12 @@ describe LaunchDarkly::Requestor do
         allow(faraday).to receive(:get) do |*args, &block|
           req = double(Faraday::Request, :headers => {}, :options => Faraday::RequestOptions.new)
           block.call(req)
-          expect(args).to eq ['http://ld.com/sdk/latest-flags']
+          expect(args).to eq ['http://ld.com/sdk/latest-all']
           expect(req.options.proxy[:uri]).to eq URI("http://proxy.com")
           double(body: '{"foo": "bar"}', status: 200, headers: {})
         end
 
-        requestor.request_all_flags()
+        requestor.request_all_data()
       end
     end
     describe "without a proxy" do
@@ -42,11 +42,11 @@ describe LaunchDarkly::Requestor do
         allow(faraday).to receive(:get) do |*args, &block|
           req = double(Faraday::Request, :headers => {}, :options => Faraday::RequestOptions.new)
           block.call(req)
-          expect(args).to eq ['http://ld.com/sdk/latest-flags']
+          expect(args).to eq ['http://ld.com/sdk/latest-all']
           expect(req.options.proxy).to eq nil
           double(body: '{"foo": "bar"}', status: 200, headers: {})
         end
-        requestor.request_all_flags()
+        requestor.request_all_data()
       end
     end
   end

data/spec/stream_spec.rb

@@ -1,42 +1,10 @@
 require "spec_helper"
 require 'ostruct'
 
-describe LaunchDarkly::InMemoryFeatureStore do
-  subject { LaunchDarkly::InMemoryFeatureStore }
-
-  include LaunchDarkly
-  
-  let(:store) { subject.new }
-  let(:key) { :asdf }
-  let(:feature) { { key: "asdf", value: "qwer", version: 0 } }
-
-  describe '#all' do
-    it "will get all keys" do
-      store.upsert(LaunchDarkly::FEATURES, feature)
-      data = store.all(LaunchDarkly::FEATURES)
-      expect(data).to eq(key => feature)
-    end
-    it "will not get deleted keys" do
-      store.upsert(LaunchDarkly::FEATURES, feature)
-      store.delete(LaunchDarkly::FEATURES, key, 1)
-      data = store.all(LaunchDarkly::FEATURES)
-      expect(data).to eq({})
-    end
-  end
-
-  describe '#initialized?' do
-    it "will return whether the store has been initialized" do
-      expect(store.initialized?).to eq false
-      store.init(key => feature)
-      expect(store.initialized?).to eq true
-    end
-  end
-end
-
 describe LaunchDarkly::StreamProcessor do
   subject { LaunchDarkly::StreamProcessor }
   let(:config) { LaunchDarkly::Config.new }
-  let(:requestor) { LaunchDarkly::Requestor.new("sdk_key", config)}
+  let(:requestor) { double() }
   let(:processor) { subject.new("sdk_key", config, requestor) }
 
   describe '#process_message' do
@@ -45,6 +13,9 @@ describe LaunchDarkly::StreamProcessor do
     let(:patch_seg_message) { OpenStruct.new({data: '{"path": "/segments/key", "data": {"key": "asdf", "version": 1}}'}) }
     let(:delete_flag_message) { OpenStruct.new({data: '{"path": "/flags/key", "version": 2}'}) }
     let(:delete_seg_message) { OpenStruct.new({data: '{"path": "/segments/key", "version": 2}'}) }
+    let(:indirect_patch_flag_message) { OpenStruct.new({data: "/flags/key"}) }
+    let(:indirect_patch_segment_message) { OpenStruct.new({data: "/segments/key"}) }
+
     it "will accept PUT methods" do
       processor.send(:process_message, put_message, LaunchDarkly::PUT)
       expect(config.feature_store.get(LaunchDarkly::FEATURES, "asdf")).to eq(key: "asdf")
@@ -68,6 +39,18 @@ describe LaunchDarkly::StreamProcessor do
       processor.send(:process_message, delete_seg_message, LaunchDarkly::DELETE)
       expect(config.feature_store.get(LaunchDarkly::SEGMENTS, "key")).to eq(nil)
     end
+    it "will accept INDIRECT PATCH method for flags" do
+      flag = { key: 'key', version: 1 }
+      allow(requestor).to receive(:request_flag).with(flag[:key]).and_return(flag)
+      processor.send(:process_message, indirect_patch_flag_message, LaunchDarkly::INDIRECT_PATCH);
+      expect(config.feature_store.get(LaunchDarkly::FEATURES, flag[:key])).to eq(flag)
+    end
+    it "will accept INDIRECT PATCH method for segments" do
+      segment = { key: 'key', version: 1 }
+      allow(requestor).to receive(:request_segment).with(segment[:key]).and_return(segment)
+      processor.send(:process_message, indirect_patch_segment_message, LaunchDarkly::INDIRECT_PATCH);
+      expect(config.feature_store.get(LaunchDarkly::SEGMENTS, segment[:key])).to eq(segment)
+    end
     it "will log a warning if the method is not recognized" do
       expect(processor.instance_variable_get(:@config).logger).to receive :warn
       processor.send(:process_message, put_message, "get")

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ldclient-rb
 version: !ruby/object:Gem::Version
-  version: 3.0.0
+  version: 3.0.1
 platform: ruby
 authors:
 - LaunchDarkly
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-02-22 00:00:00.000000000 Z
+date: 2018-02-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler