diff options
| author | Rémy Coutable <remy@rymai.me> | 2018-04-19 18:53:07 +0200 |
|---|---|---|
| committer | Rémy Coutable <remy@rymai.me> | 2018-05-18 12:25:54 +0200 |
| commit | 2ad01c5ab079959205c236df7bf8cbb5c1fab573 (patch) | |
| tree | 3151e48d210b853f7c65dafc2fa0f4069ea442fc /config/initializers | |
| parent | 9f863dbe1a5ed324b7fb202eea11397d1fcdd61f (diff) | |
| download | gitlab-ce-2ad01c5ab079959205c236df7bf8cbb5c1fab573.tar.gz | |
Ensure Flipper memoizer is used in Sidekiq's context45528-repeated-calls-to-redis-for-flipper-feature-flag
Also, don't use the provided Middleware, which isn't thread-safe, and
instantiate a new Flipper instance per thread instead.
Signed-off-by: Rémy Coutable <remy@rymai.me>
Diffstat (limited to 'config/initializers')
| -rw-r--r-- | config/initializers/flipper.rb | 21 |
1 files changed, 0 insertions, 21 deletions
diff --git a/config/initializers/flipper.rb b/config/initializers/flipper.rb index c60ad535fd5..80cab7273e5 100644 --- a/config/initializers/flipper.rb +++ b/config/initializers/flipper.rb @@ -1,22 +1 @@ -require 'flipper/adapters/active_record' -require 'flipper/adapters/active_support_cache_store' - -Flipper.configure do |config| - config.default do - adapter = Flipper::Adapters::ActiveRecord.new( - feature_class: Feature::FlipperFeature, gate_class: Feature::FlipperGate) - cached_adapter = Flipper::Adapters::ActiveSupportCacheStore.new( - adapter, - Rails.cache, - expires_in: 1.hour) - - Flipper.new(cached_adapter) - end -end - Feature.register_feature_groups - -unless Rails.env.test? - require 'flipper/middleware/memoizer' - Rails.application.config.middleware.use Flipper::Middleware::Memoizer -end |