flipper-cloud 0.20.2 → 0.21.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ad99886efa608844beb4f7efdd5603ff60280c2ac5ca871a8de2e5fcbb2459f5
-  data.tar.gz: f5f162782dd28d9d5e55616030202eb47899438f700d12ec7de218bc9bdba353
+  metadata.gz: 8f8ef3d6224837583368a9c10fd382244f493e57fe324a3c5e9bbeb2d8ed16c7
+  data.tar.gz: 0a21cf517ac2e92b791f9eb6436ee123af93202ade8f0d815af0103bd9c07606
 SHA512:
-  metadata.gz: 44632f5b5c82b6c72c447716f8946768c03d2388c60fbfcdc07b43ee79389f094d49da3c1a1b5c4f34b4a1172e72b1dd2dbc67128838210f9cb3fb0dace41193
-  data.tar.gz: d6968fb80d7cdc47ebb5bb692780e6eee715faf845644302c28cc5f86217c19c939df18456382e5e614d9a72f39d3793298d054aa7d8d076d143ae3393eee8fc
+  metadata.gz: b2e5e4ae41036280248bc5edf3a569752a35ce4a053464c2d0955e45d7c39172ad4af089a7242924547f073f7b15ac0cb8ebcfb33e8f175db374623663ebf3a1
+  data.tar.gz: 8b4666ff117ed3ff2ac70fc726f19ccae58af60479a7d596cf01e672ec1e1b8e1efa370ba1e08debbfc3378e3a12565499be509aa3ddc682da304c9ff5fa7bed

data/examples/cloud/app.ru

@@ -1,14 +1,9 @@
 # Usage (from the repo root):
-#   env FLIPPER_CLOUD_TOKEN=<token> FLIPPER_CLOUD_SYNC_SECRET=<secret> FLIPPER_CLOUD_SYNC_METHOD=webhook bundle exec rackup examples/ui/basic.ru -p 9999
-#   env FLIPPER_CLOUD_TOKEN=<token> FLIPPER_CLOUD_SYNC_SECRET=<secret> FLIPPER_CLOUD_SYNC_METHOD=webhook bundle exec shotgun examples/ui/basic.ru -p 9999
+#   env FLIPPER_CLOUD_TOKEN=<token> FLIPPER_CLOUD_SYNC_SECRET=<secret> bundle exec rackup examples/cloud/app.ru -p 9999
+#   env FLIPPER_CLOUD_TOKEN=<token> FLIPPER_CLOUD_SYNC_SECRET=<secret> bundle exec shotgun examples/cloud/app.ru -p 9999
 #   http://localhost:9999/
-#   http://localhost:9999/webhooks
-
-require 'pathname'
-root_path = Pathname(__FILE__).dirname.join('..').expand_path
-lib_path  = root_path.join('lib')
-$:.unshift(lib_path)
 
+require 'bundler/setup'
 require 'flipper/cloud'
 Flipper.configure do |config|
   config.default { Flipper::Cloud.new }

data/examples/cloud/basic.rb

@@ -1,25 +1,19 @@
 # Usage (from the repo root):
 # env FLIPPER_CLOUD_TOKEN=<token> bundle exec ruby examples/cloud/basic.rb
-require 'pathname'
-require 'logger'
-root_path = Pathname(__FILE__).dirname.join('..').expand_path
-lib_path  = root_path.join('lib')
-$:.unshift(lib_path)
-
+require 'bundler/setup'
 require 'flipper/cloud'
-flipper = Flipper::Cloud.new
 
-flipper[:stats].enable
+Flipper[:stats].enable
 
-if flipper[:stats].enabled?
+if Flipper[:stats].enabled?
   puts 'Enabled!'
 else
   puts 'Disabled!'
 end
 
-flipper[:stats].disable
+Flipper[:stats].disable
 
-if flipper[:stats].enabled?
+if Flipper[:stats].enabled?
   puts 'Enabled!'
 else
   puts 'Disabled!'

data/examples/cloud/import.rb

@@ -1,23 +1,15 @@
 # Usage (from the repo root):
 #   env FLIPPER_CLOUD_TOKEN=<token> bundle exec ruby examples/cloud/import.rb
-require 'pathname'
-require 'logger'
-root_path = Pathname(__FILE__).dirname.join('..').expand_path
-lib_path  = root_path.join('lib')
-$:.unshift(lib_path)
-
+require 'bundler/setup'
 require 'flipper'
 require 'flipper/cloud'
 
-memory_adapter = Flipper::Adapters::Memory.new
-flipper = Flipper.new(memory_adapter)
-
-flipper.enable(:test)
-flipper.enable(:search)
-flipper.enable_actor(:stats, Flipper::Actor.new("jnunemaker"))
-flipper.enable_percentage_of_time(:logging, 5)
+Flipper.enable(:test)
+Flipper.enable(:search)
+Flipper.enable_actor(:stats, Flipper::Actor.new("jnunemaker"))
+Flipper.enable_percentage_of_time(:logging, 5)
 
 cloud = Flipper::Cloud.new
 
 # makes cloud identical to memory flipper
-cloud.import(flipper)
+cloud.import(Flipper)

data/lib/flipper/cloud.rb

@@ -4,6 +4,7 @@ require "flipper/middleware/memoizer"
 require "flipper/cloud/configuration"
 require "flipper/cloud/dsl"
 require "flipper/cloud/middleware"
+require "flipper/cloud/engine" if defined?(Rails::Engine)
 
 module Flipper
   module Cloud
@@ -14,8 +15,14 @@ module Flipper
     # options - The Hash of options. See Flipper::Cloud::Configuration.
     # block - The block that configuration will be yielded to allowing you to
     #         customize this cloud instance and its adapter.
-    def self.new(token = nil, options = {})
-      options = options.merge(token: token) if token
+    def self.new(options = {}, deprecated_options = {})
+      if options.is_a?(String)
+        warn "`Flipper::Cloud.new(token)` is deprecated. Use `Flipper::Cloud.new(token: token)` " +
+          "or set the `FLIPPER_CLOUD_TOKEN` environment variable.\n" +
+          caller[0]
+        options = deprecated_options.merge(token: options)
+      end
+
       configuration = Configuration.new(options)
       yield configuration if block_given?
       DSL.new(configuration)
@@ -36,5 +43,21 @@ module Flipper
       builder.define_singleton_method(:inspect) { klass.inspect } # pretty rake routes output
       builder
     end
+
+    # Private: Configure Flipper to use Cloud by default
+    def self.set_default
+      Flipper.configure do |config|
+        config.default do
+          if ENV["FLIPPER_CLOUD_TOKEN"]
+            self.new(local_adapter: config.adapter)
+          else
+            warn "Missing FLIPPER_CLOUD_TOKEN environment variable. Disabling Flipper::Cloud."
+            Flipper.new(config.adapter)
+          end
+        end
+      end
+    end
   end
 end
+
+Flipper::Cloud.set_default

data/lib/flipper/cloud/configuration.rb

@@ -13,6 +13,8 @@ module Flipper
         :webhook,
       ].freeze
 
+      DEFAULT_URL = "https://www.flippercloud.io/adapter".freeze
+
       # Public: The token corresponding to an environment on flippercloud.io.
       attr_accessor :token
 
@@ -57,10 +59,6 @@ module Flipper
       # the local in sync with cloud (default: 10).
       attr_accessor :sync_interval
 
-      # Public: The method to be used for synchronizing your local flipper
-      # adapter with cloud. (default: :poll, can also be :webhook).
-      attr_reader :sync_method
-
       # Public: The secret used to verify if syncs in the middleware should
       # occur or not.
       attr_accessor :sync_secret
@@ -69,9 +67,14 @@ module Flipper
         @token = options.fetch(:token) { ENV["FLIPPER_CLOUD_TOKEN"] }
 
         if @token.nil?
-          raise ArgumentError, "Flipper::Cloud token is missing. Please set FLIPPER_CLOUD_TOKEN or provide the token (e.g. Flipper::Cloud.new('token'))."
+          raise ArgumentError, "Flipper::Cloud token is missing. Please set FLIPPER_CLOUD_TOKEN or provide the token (e.g. Flipper::Cloud.new(token: 'token'))."
         end
 
+        if ENV["FLIPPER_CLOUD_SYNC_METHOD"]
+          warn "FLIPPER_CLOUD_SYNC_METHOD is deprecated and has no effect."
+        end
+        self.sync_method = options[:sync_method] if options[:sync_method]
+
         @instrumenter = options.fetch(:instrumenter, Instrumenters::Noop)
         @read_timeout = options.fetch(:read_timeout) { ENV.fetch("FLIPPER_CLOUD_READ_TIMEOUT", 5).to_f }
         @open_timeout = options.fetch(:open_timeout) { ENV.fetch("FLIPPER_CLOUD_OPEN_TIMEOUT", 5).to_f }
@@ -81,8 +84,7 @@ module Flipper
         @local_adapter = options.fetch(:local_adapter) { Adapters::Memory.new }
         @debug_output = options[:debug_output]
         @adapter_block = ->(adapter) { adapter }
-        self.sync_method = options.fetch(:sync_method) { ENV.fetch("FLIPPER_CLOUD_SYNC_METHOD", :poll).to_sym }
-        self.url = options.fetch(:url) { ENV.fetch("FLIPPER_CLOUD_URL", "https://www.flippercloud.io/adapter".freeze) }
+        self.url = options.fetch(:url) { ENV.fetch("FLIPPER_CLOUD_URL", DEFAULT_URL) }
       end
 
       # Public: Read or customize the http adapter. Calling without a block will
@@ -114,18 +116,14 @@ module Flipper
         }).call
       end
 
-      def sync_method=(new_sync_method)
-        new_sync_method = new_sync_method.to_sym
-
-        unless VALID_SYNC_METHODS.include?(new_sync_method)
-          raise ArgumentError, "Unsupported sync_method. Valid options are (#{VALID_SYNC_METHODS.to_a.join(', ')})"
-        end
-
-        if new_sync_method == :webhook && sync_secret.nil?
-          raise ArgumentError, "Flipper::Cloud sync_secret is missing. Please set FLIPPER_CLOUD_SYNC_SECRET or provide the sync_secret used to validate webhooks."
-        end
+      # Public: The method that will be used to synchronize local adapter with
+      # cloud. (default: :poll, will be :webhook if sync_secret is set).
+      def sync_method
+        sync_secret ? :webhook : :poll
+      end
 
-        @sync_method = new_sync_method
+      def sync_method=(_)
+        warn "Flipper::Cloud: sync_method is deprecated and has no effect."
       end
 
       private

data/lib/flipper/cloud/engine.rb

@@ -0,0 +1,29 @@
+require "flipper/railtie"
+
+module Flipper
+  module Cloud
+    class Engine < Rails::Engine
+      paths["config/routes.rb"] = ["lib/flipper/cloud/routes.rb"]
+
+      config.before_configuration do
+        config.flipper.cloud_path = "_flipper"
+      end
+
+      initializer "flipper.cloud.default", before: :load_config_initializers do |app|
+        Flipper.configure do |config|
+          config.default do
+            if ENV["FLIPPER_CLOUD_TOKEN"]
+              Flipper::Cloud.new(
+                local_adapter: config.adapter,
+                instrumenter: app.config.flipper.instrumenter
+              )
+            else
+              warn "Missing FLIPPER_CLOUD_TOKEN environment variable. Disabling Flipper::Cloud."
+              Flipper.new(config.adapter)
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/flipper/cloud/middleware.rb

@@ -7,6 +7,8 @@ module Flipper
     class Middleware
       # Internal: The path to match for webhook requests.
       WEBHOOK_PATH = %r{\A/webhooks\/?\Z}
+      # Internal: The root path to match for requests.
+      ROOT_PATH = %r{\A/\Z}
 
       def initialize(app, options = {})
         @app = app
@@ -19,7 +21,7 @@ module Flipper
 
       def call!(env)
         request = Rack::Request.new(env)
-        if request.post? && request.path_info.match(WEBHOOK_PATH)
+        if request.post? && (request.path_info.match(ROOT_PATH) || request.path_info.match(WEBHOOK_PATH))
           status = 200
           headers = {
             "Content-Type" => "application/json",

data/lib/flipper/cloud/routes.rb

@@ -0,0 +1,13 @@
+# Default routes loaded by Flipper::Cloud::Engine
+Rails.application.routes.draw do
+  if ENV["FLIPPER_CLOUD_TOKEN"] && ENV["FLIPPER_CLOUD_SYNC_SECRET"]
+    config = Rails.application.config.flipper
+
+    cloud_app = Flipper::Cloud.app(nil,
+      env_key: config.env_key,
+      memoizer_options: { preload: config.preload }
+    )
+
+    mount cloud_app, at: config.cloud_path
+  end
+end

data/lib/flipper/version.rb

@@ -1,3 +1,3 @@
 module Flipper
-  VERSION = '0.20.2'.freeze
+  VERSION = '0.21.0'.freeze
 end

data/spec/flipper/cloud/configuration_spec.rb

@@ -127,53 +127,30 @@ RSpec.describe Flipper::Cloud::Configuration do
     end
   end
 
-  it "defaults to sync_method to poll" do
-    memory_adapter = Flipper::Adapters::Memory.new
+  it "defaults sync_method to :poll" do
     instance = described_class.new(required_options)
 
     expect(instance.sync_method).to eq(:poll)
   end
 
-  it "can use webhook for sync_method" do
-    memory_adapter = Flipper::Adapters::Memory.new
+  it "sets sync_method to :webhook if sync_secret provided" do
     instance = described_class.new(required_options.merge({
       sync_secret: "secret",
-      sync_method: :webhook,
-      local_adapter: memory_adapter,
     }))
 
     expect(instance.sync_method).to eq(:webhook)
     expect(instance.adapter).to be_instance_of(Flipper::Adapters::DualWrite)
   end
 
-  it "raises ArgumentError for invalid sync_method" do
-    expect {
-      described_class.new(required_options.merge(sync_method: :foo))
-    }.to raise_error(ArgumentError, "Unsupported sync_method. Valid options are (poll, webhook)")
-  end
-
-  it "can use ENV var for sync_method" do
-    with_modified_env "FLIPPER_CLOUD_SYNC_METHOD" => "webhook" do
-      instance = described_class.new(required_options.merge({
-        sync_secret: "secret",
-      }))
+  it "sets sync_method to :webhook if FLIPPER_CLOUD_SYNC_SECRET set" do
+    with_modified_env "FLIPPER_CLOUD_SYNC_SECRET" => "abc" do
+      instance = described_class.new(required_options)
 
       expect(instance.sync_method).to eq(:webhook)
+      expect(instance.adapter).to be_instance_of(Flipper::Adapters::DualWrite)
     end
   end
 
-  it "can use string sync_method instead of symbol" do
-    memory_adapter = Flipper::Adapters::Memory.new
-    instance = described_class.new(required_options.merge({
-      sync_secret: "secret",
-      sync_method: "webhook",
-      local_adapter: memory_adapter,
-    }))
-
-    expect(instance.sync_method).to eq(:webhook)
-    expect(instance.adapter).to be_instance_of(Flipper::Adapters::DualWrite)
-  end
-
   it "can set sync_secret" do
     instance = described_class.new(required_options.merge(sync_secret: "from_config"))
       expect(instance.sync_secret).to eq("from_config")

data/spec/flipper/cloud/dsl_spec.rb

@@ -9,7 +9,6 @@ RSpec.describe Flipper::Cloud::DSL do
     cloud_configuration = Flipper::Cloud::Configuration.new({
       token: "asdf",
       sync_secret: "tasty",
-      sync_method: :webhook,
     })
     dsl = described_class.new(cloud_configuration)
     expect(dsl.features).to eq(Set.new)
@@ -26,7 +25,6 @@ RSpec.describe Flipper::Cloud::DSL do
     cloud_configuration = Flipper::Cloud::Configuration.new({
       token: "asdf",
       sync_secret: "tasty",
-      sync_method: :webhook,
     })
     dsl = described_class.new(cloud_configuration)
     dsl.sync
@@ -37,7 +35,6 @@ RSpec.describe Flipper::Cloud::DSL do
     cloud_configuration = Flipper::Cloud::Configuration.new({
       token: "asdf",
       sync_secret: "tasty",
-      sync_method: :webhook,
     })
     dsl = described_class.new(cloud_configuration)
     expect(dsl.sync_secret).to eq("tasty")
@@ -52,7 +49,6 @@ RSpec.describe Flipper::Cloud::DSL do
       cloud_configuration = Flipper::Cloud::Configuration.new({
         token: "asdf",
         sync_secret: "tasty",
-        sync_method: :webhook,
         local_adapter: local_adapter
       })
     end

data/spec/flipper/cloud/engine_spec.rb

@@ -0,0 +1,98 @@
+require 'helper'
+require 'rails'
+require 'flipper/cloud'
+
+RSpec.describe Flipper::Cloud::Engine do
+  let(:env) do
+    { "FLIPPER_CLOUD_TOKEN" => "test-token" }
+  end
+
+  let(:application) do
+    Class.new(Rails::Application) do
+      config.eager_load = false
+      config.logger = ActiveSupport::Logger.new($stdout)
+    end
+  end
+
+  # App for Rack::Test
+  let(:app) { application.routes }
+
+  before do
+    Rails.application = nil
+
+    # Force loading of flipper to configure itself
+    load 'flipper/cloud.rb'
+  end
+
+  it "initializes cloud configuration" do
+    stub_request(:get, /flippercloud\.io/).to_return(status: 200, body: "{}")
+
+    with_modified_env env do
+      application.initialize!
+
+      expect(Flipper.instance).to be_a(Flipper::Cloud::DSL)
+      expect(Flipper.instance.instrumenter).to be(ActiveSupport::Notifications)
+    end
+  end
+
+  context "with CLOUD_SYNC_SECRET" do
+    before do
+      env.update "FLIPPER_CLOUD_SYNC_SECRET" => "test-secret"
+    end
+
+    let(:request_body) do
+      JSON.generate({
+        "environment_id" => 1,
+        "webhook_id" => 1,
+        "delivery_id" => SecureRandom.uuid,
+        "action" => "sync",
+      })
+    end
+    let(:timestamp) { Time.now }
+    let(:signature) {
+      Flipper::Cloud::MessageVerifier.new(secret: env["FLIPPER_CLOUD_SYNC_SECRET"]).generate(request_body, timestamp)
+    }
+    let(:signature_header_value) {
+      Flipper::Cloud::MessageVerifier.new(secret: "").header(signature, timestamp)
+    }
+
+    it "configures webhook app" do
+      with_modified_env env do
+        application.initialize!
+
+        stub = stub_request(:get, "https://www.flippercloud.io/adapter/features").with({
+          headers: { "Flipper-Cloud-Token" => ENV["FLIPPER_CLOUD_TOKEN"] },
+        }).to_return(status: 200, body: JSON.generate({ features: {} }), headers: {})
+
+        post "/_flipper", request_body, { "HTTP_FLIPPER_CLOUD_SIGNATURE" => signature_header_value }
+
+        expect(last_response.status).to eq(200)
+        expect(stub).to have_been_requested
+      end
+    end
+  end
+
+  context "without CLOUD_SYNC_SECRET" do
+    it "does not configure webhook app" do
+      with_modified_env env do
+        application.initialize!
+
+        post "/_flipper"
+        expect(last_response.status).to eq(404)
+      end
+    end
+  end
+
+  context "without FLIPPER_CLOUD_TOKEN" do
+    it "gracefully skips configuring webhook app" do
+      with_modified_env "FLIPPER_CLOUD_TOKEN" => nil do
+        application.initialize!
+        expect(silence { Flipper.instance }).to match(/Missing FLIPPER_CLOUD_TOKEN/)
+        expect(Flipper.instance).to be_a(Flipper::DSL)
+
+        post "/_flipper"
+        expect(last_response.status).to eq(404)
+      end
+    end
+  end
+end

data/spec/flipper/cloud/middleware_spec.rb

@@ -6,18 +6,16 @@ require 'flipper/adapters/operation_logger'
 
 RSpec.describe Flipper::Cloud::Middleware do
   let(:flipper) {
-    Flipper::Cloud.new("regular") do |config|
+    Flipper::Cloud.new(token: "regular") do |config|
       config.local_adapter = Flipper::Adapters::OperationLogger.new(Flipper::Adapters::Memory.new)
       config.sync_secret = "regular_tasty"
-      config.sync_method = :webhook
     end
   }
 
   let(:env_flipper) {
-    Flipper::Cloud.new("env") do |config|
+    Flipper::Cloud.new(token: "env") do |config|
       config.local_adapter = Flipper::Adapters::OperationLogger.new(Flipper::Adapters::Memory.new)
       config.sync_secret = "env_tasty"
-      config.sync_method = :webhook
     end
   }
 
@@ -53,7 +51,7 @@ RSpec.describe Flipper::Cloud::Middleware do
       env = {
         "HTTP_FLIPPER_CLOUD_SIGNATURE" => signature_header_value,
       }
-      post '/webhooks', request_body, env
+      post '/', request_body, env
 
       expect(last_response.status).to eq(200)
       expect(JSON.parse(last_response.body)).to eq({
@@ -80,7 +78,7 @@ RSpec.describe Flipper::Cloud::Middleware do
       env = {
         "HTTP_FLIPPER_CLOUD_SIGNATURE" => signature_header_value,
       }
-      post '/webhooks', request_body, env
+      post '/', request_body, env
 
       expect(last_response.status).to eq(400)
       expect(stub).not_to have_been_requested
@@ -101,11 +99,11 @@ RSpec.describe Flipper::Cloud::Middleware do
       env = {
         "HTTP_FLIPPER_CLOUD_SIGNATURE" => signature_header_value,
       }
-      post '/webhooks', request_body, env
+      post '/', request_body, env
 
       expect(last_response.status).to eq(402)
       expect(last_response.headers["Flipper-Cloud-Response-Error-Class"]).to eq("Flipper::Adapters::Http::Error")
-      expect(last_response.headers["Flipper-Cloud-Response-Error-Message"]).to eq("Failed with status: 402")
+      expect(last_response.headers["Flipper-Cloud-Response-Error-Message"]).to include("Failed with status: 402")
       expect(stub).to have_been_requested
     end
   end
@@ -124,11 +122,11 @@ RSpec.describe Flipper::Cloud::Middleware do
       env = {
         "HTTP_FLIPPER_CLOUD_SIGNATURE" => signature_header_value,
       }
-      post '/webhooks', request_body, env
+      post '/', request_body, env
 
       expect(last_response.status).to eq(500)
       expect(last_response.headers["Flipper-Cloud-Response-Error-Class"]).to eq("Flipper::Adapters::Http::Error")
-      expect(last_response.headers["Flipper-Cloud-Response-Error-Message"]).to eq("Failed with status: 503")
+      expect(last_response.headers["Flipper-Cloud-Response-Error-Message"]).to include("Failed with status: 503")
       expect(stub).to have_been_requested
     end
   end
@@ -147,7 +145,7 @@ RSpec.describe Flipper::Cloud::Middleware do
       env = {
         "HTTP_FLIPPER_CLOUD_SIGNATURE" => signature_header_value,
       }
-      post '/webhooks', request_body, env
+      post '/', request_body, env
 
       expect(last_response.status).to eq(500)
       expect(last_response.headers["Flipper-Cloud-Response-Error-Class"]).to eq("Net::OpenTimeout")
@@ -168,7 +166,7 @@ RSpec.describe Flipper::Cloud::Middleware do
         "HTTP_FLIPPER_CLOUD_SIGNATURE" => signature_header_value,
         'flipper' => env_flipper,
       }
-      post '/webhooks', request_body, env
+      post '/', request_body, env
 
       expect(last_response.status).to eq(200)
       expect(stub).to have_been_requested
@@ -187,7 +185,7 @@ RSpec.describe Flipper::Cloud::Middleware do
         "HTTP_FLIPPER_CLOUD_SIGNATURE" => signature_header_value,
         'flipper' => env_flipper,
       }
-      post '/webhooks', request_body, env
+      post '/', request_body, env
 
       expect(last_response.status).to eq(200)
       expect(stub).to have_been_requested
@@ -207,7 +205,7 @@ RSpec.describe Flipper::Cloud::Middleware do
         'flipper' => flipper,
         'flipper_cloud' => env_flipper,
       }
-      post '/webhooks', request_body, env
+      post '/', request_body, env
 
       expect(last_response.status).to eq(200)
       expect(stub).to have_been_requested
@@ -218,6 +216,27 @@ RSpec.describe Flipper::Cloud::Middleware do
     let(:app) { Flipper::Cloud.app(-> { flipper }) }
 
     it 'works' do
+      stub = stub_request_for_token('regular')
+      env = {
+        "HTTP_FLIPPER_CLOUD_SIGNATURE" => signature_header_value,
+      }
+      post '/', request_body, env
+
+      expect(last_response.status).to eq(200)
+      expect(stub).to have_been_requested
+    end
+  end
+
+  context 'when using older /webhooks path' do
+    let(:app) { Flipper::Cloud.app(flipper) }
+
+    it 'uses instance to sync' do
+      Flipper.register(:admins) { |*args| false }
+      Flipper.register(:staff) { |*args| false }
+      Flipper.register(:basic) { |*args| false }
+      Flipper.register(:plus) { |*args| false }
+      Flipper.register(:premium) { |*args| false }
+
       stub = stub_request_for_token('regular')
       env = {
         "HTTP_FLIPPER_CLOUD_SIGNATURE" => signature_header_value,
@@ -225,13 +244,22 @@ RSpec.describe Flipper::Cloud::Middleware do
       post '/webhooks', request_body, env
 
       expect(last_response.status).to eq(200)
+      expect(JSON.parse(last_response.body)).to eq({
+        "groups" => [
+          {"name" => "admins"},
+          {"name" => "staff"},
+          {"name" => "basic"},
+          {"name" => "plus"},
+          {"name" => "premium"},
+        ],
+      })
       expect(stub).to have_been_requested
     end
   end
 
   describe 'Request method unsupported' do
     it 'skips middleware' do
-      get '/webhooks'
+      get '/'
       expect(last_response.status).to eq(404)
       expect(last_response.content_type).to eq("application/json")
       expect(last_response.body).to eq("{}")

data/spec/flipper/cloud_spec.rb

@@ -12,7 +12,7 @@ RSpec.describe Flipper::Cloud do
     let(:token) { 'asdf' }
 
     before do
-      @instance = described_class.new(token)
+      @instance = described_class.new(token: token)
       memoized_adapter = @instance.adapter
       sync_adapter = memoized_adapter.adapter
       @http_adapter = sync_adapter.instance_variable_get('@remote')
@@ -52,7 +52,7 @@ RSpec.describe Flipper::Cloud do
     before do
       stub_request(:get, /fakeflipper\.com/).to_return(status: 200, body: "{}")
 
-      @instance = described_class.new('asdf', url: 'https://www.fakeflipper.com/sadpanda')
+      @instance = described_class.new(token: 'asdf', url: 'https://www.fakeflipper.com/sadpanda')
       memoized_adapter = @instance.adapter
       sync_adapter = memoized_adapter.adapter
       @http_adapter = sync_adapter.instance_variable_get('@remote')
@@ -75,12 +75,12 @@ RSpec.describe Flipper::Cloud do
 
   it 'can set instrumenter' do
     instrumenter = Flipper::Instrumenters::Memory.new
-    instance = described_class.new('asdf', instrumenter: instrumenter)
+    instance = described_class.new(token: 'asdf', instrumenter: instrumenter)
     expect(instance.instrumenter).to be(instrumenter)
   end
 
   it 'allows wrapping adapter with another adapter like the instrumenter' do
-    instance = described_class.new('asdf') do |config|
+    instance = described_class.new(token: 'asdf') do |config|
       config.adapter do |adapter|
         Flipper::Adapters::Instrumented.new(adapter)
       end
@@ -92,26 +92,26 @@ RSpec.describe Flipper::Cloud do
   it 'can set debug_output' do
     expect(Flipper::Adapters::Http::Client).to receive(:new)
       .with(hash_including(debug_output: STDOUT))
-    described_class.new('asdf', debug_output: STDOUT)
+    described_class.new(token: 'asdf', debug_output: STDOUT)
   end
 
   it 'can set read_timeout' do
     expect(Flipper::Adapters::Http::Client).to receive(:new)
       .with(hash_including(read_timeout: 1))
-    described_class.new('asdf', read_timeout: 1)
+    described_class.new(token: 'asdf', read_timeout: 1)
   end
 
   it 'can set open_timeout' do
     expect(Flipper::Adapters::Http::Client).to receive(:new)
       .with(hash_including(open_timeout: 1))
-    described_class.new('asdf', open_timeout: 1)
+    described_class.new(token: 'asdf', open_timeout: 1)
   end
 
   if RUBY_VERSION >= '2.6.0'
     it 'can set write_timeout' do
       expect(Flipper::Adapters::Http::Client).to receive(:new)
         .with(hash_including(open_timeout: 1))
-      described_class.new('asdf', open_timeout: 1)
+      described_class.new(token: 'asdf', open_timeout: 1)
     end
   end
 
@@ -129,7 +129,7 @@ RSpec.describe Flipper::Cloud do
     flipper.enable_actor(:stats, Flipper::Actor.new("jnunemaker"))
     flipper.enable_percentage_of_time(:logging, 5)
 
-    cloud_flipper = Flipper::Cloud.new("asdf")
+    cloud_flipper = Flipper::Cloud.new(token: "asdf")
 
     get_all = {
       "logging" => {actors: Set.new, boolean: nil, groups: Set.new, percentage_of_actors: nil, percentage_of_time: "5"},
@@ -158,7 +158,7 @@ RSpec.describe Flipper::Cloud do
     flipper.enable_actor(:stats, Flipper::Actor.new("jnunemaker"))
     flipper.enable_percentage_of_time(:logging, 5)
 
-    cloud_flipper = Flipper::Cloud.new("asdf")
+    cloud_flipper = Flipper::Cloud.new(token: "asdf")
 
     get_all = {
       "logging" => {actors: Set.new, boolean: nil, groups: Set.new, percentage_of_actors: nil, percentage_of_time: "5"},
@@ -186,7 +186,7 @@ RSpec.describe Flipper::Cloud do
     flipper.enable_actor(:stats, Flipper::Actor.new("jnunemaker"))
     flipper.enable_percentage_of_time(:logging, 5)
 
-    cloud_flipper = Flipper::Cloud.new("asdf")
+    cloud_flipper = Flipper::Cloud.new(token: "asdf")
 
     get_all = {
       "logging" => {actors: Set.new, boolean: nil, groups: Set.new, percentage_of_actors: nil, percentage_of_time: "5"},

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: flipper-cloud
 version: !ruby/object:Gem::Version
-  version: 0.20.2
+  version: 0.21.0
 platform: ruby
 authors:
 - John Nunemaker
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-01-08 00:00:00.000000000 Z
+date: 2021-05-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: flipper
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.20.2
+        version: 0.21.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.20.2
+        version: 0.21.0
 description: 
 email:
 - nunemaker@gmail.com
@@ -31,21 +31,23 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- docs/images/flipper_cloud.png
 - examples/cloud/app.ru
 - examples/cloud/basic.rb
-- examples/cloud/cached_in_memory.rb
 - examples/cloud/import.rb
-- examples/cloud/local_adapter.rb
 - flipper-cloud.gemspec
 - lib/flipper-cloud.rb
 - lib/flipper/cloud.rb
 - lib/flipper/cloud/configuration.rb
 - lib/flipper/cloud/dsl.rb
+- lib/flipper/cloud/engine.rb
 - lib/flipper/cloud/message_verifier.rb
 - lib/flipper/cloud/middleware.rb
+- lib/flipper/cloud/routes.rb
 - lib/flipper/version.rb
 - spec/flipper/cloud/configuration_spec.rb
 - spec/flipper/cloud/dsl_spec.rb
+- spec/flipper/cloud/engine_spec.rb
 - spec/flipper/cloud/message_verifier_spec.rb
 - spec/flipper/cloud/middleware_spec.rb
 - spec/flipper/cloud_spec.rb
@@ -76,6 +78,7 @@ summary: FeatureFlipper.com adapter for Flipper
 test_files:
 - spec/flipper/cloud/configuration_spec.rb
 - spec/flipper/cloud/dsl_spec.rb
+- spec/flipper/cloud/engine_spec.rb
 - spec/flipper/cloud/message_verifier_spec.rb
 - spec/flipper/cloud/middleware_spec.rb
 - spec/flipper/cloud_spec.rb

data/examples/cloud/cached_in_memory.rb

@@ -1,29 +0,0 @@
-# env FLIPPER_CLOUD_TOKEN=<token> bundle exec ruby examples/cloud/cached_in_memory.rb
-require File.expand_path('../../example_setup', __FILE__)
-
-require 'flipper/cloud'
-require 'flipper/adapters/active_support_cache_store'
-require 'active_support/cache'
-require 'active_support/cache/memory_store'
-
-feature_name = ENV.fetch("FEATURE") { "testing" }.to_sym
-
-Flipper.configure do |config|
-  config.default do
-    Flipper::Cloud.new do |cloud|
-      cloud.debug_output = STDOUT
-      cloud.adapter do |adapter|
-        Flipper::Adapters::ActiveSupportCacheStore.new(adapter,
-          ActiveSupport::Cache::MemoryStore.new, {expires_in: 5.seconds})
-      end
-    end
-  end
-end
-
-loop do
-  # Should only print out http call every 5 seconds
-  p Flipper.enabled?(feature_name)
-  puts "\n\n"
-
-  sleep 1
-end

data/examples/cloud/local_adapter.rb

@@ -1,36 +0,0 @@
-# This is an example of using cloud with a local adapter. All cloud feature
-# changes are synced to the local adapter on an interval. All feature reads are
-# directed to the local adapter, which means reads are fast and not dependent on
-# cloud being available. You can turn internet on/off and more and this should
-# never raise. You could get a slow request every now and then if cloud is
-# unavailable, but we are hoping to fix that soon by doing the cloud update in a
-# background thread.
-# env FLIPPER_CLOUD_TOKEN=<token> bundle exec ruby examples/cloud/local_adapter.rb
-require File.expand_path('../../example_setup', __FILE__)
-
-require 'logger'
-require 'flipper/cloud'
-require 'flipper/adapters/redis'
-
-feature_name = ENV.fetch("FEATURE") { "testing" }.to_sym
-
-redis = Redis.new(logger: Logger.new(STDOUT))
-redis.flushdb
-
-Flipper.configure do |config|
-  config.default do
-    Flipper::Cloud.new do |cloud|
-      cloud.debug_output = STDOUT
-      cloud.local_adapter = Flipper::Adapters::Redis.new(redis)
-      cloud.sync_interval = 10
-    end
-  end
-end
-
-loop do
-  # Should only print out http call every 10 seconds
-  p Flipper.enabled?(feature_name)
-  puts "\n\n"
-
-  sleep 1
-end