eeny-meeny 2.1.4 → 2.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 593babb310531c408fd76d2e1d3acc142ee4f98a
-  data.tar.gz: 2cb7489e568834c7aa6ac01105d8be9fd4088fcf
+  metadata.gz: 9bdb7307a93707a324593a1cb18098053225834c
+  data.tar.gz: 8e6895a3a9c53cea18980a419fda79ee97eff00b
 SHA512:
-  metadata.gz: ba008a88abe4141df40a3685149037737a562a82b35b9ec290b93154c51a917604034d9ce73f3724e04678dd6c44573f7e2f00e2443e44738673a8a02e838471
-  data.tar.gz: 5d24cb185e1e8ea075957664f5597399c2346cfaffd0f56bdfa934e05bc7c58ffa259115306362e3fcda02840083cf82f7a68367391a108c570e250a62c13d20
+  metadata.gz: 4635a786f22f51f5f9402a7822cd061a3c3cad729448ec323834c5f21b86438fd902d4a2a180d290d25c33a097c5a65136f4c1c6f0b58cfa7632797cb6a7f600
+  data.tar.gz: '09965f4d8b1685cda659b4e31c0fa8de8b536a78e9976a7a7823b0561933ba1f639722284375e45cb6bad5452f38f199dd81b8121e6fef084ccb93b5d81feb9a'

data/CHANGELOG.md

@@ -1,3 +1,13 @@
+### 2.2.0 (2018-07-24)
+
+Features:
+
+    - Let `Eeny::Meeny::Middleware` remove deprecated experiment cookies (for undefined and inactive experiements)
+
+Changes:
+
+    - Renamed `EenyMeeny::Cookie::COOKIE_PREFIX` to `EenyMeeny::Cookie::EXPERIMENT_PREFIX`
+
 ### 2.1.4 (2017-10-04)
 
 Changes:

data/eeny-meeny.gemspec

@@ -3,7 +3,7 @@ require File.expand_path('../lib/eeny-meeny/version', __FILE__)
 Gem::Specification.new do |s|
   s.name        = 'eeny-meeny'
   s.version     = EenyMeeny::VERSION.dup
-  s.date        = '2017-10-04'
+  s.date        = '2018-07-24'
   s.summary     = "A simple split and smoke testing tool for Rails"
   s.authors     = ["Christian Orthmann"]
   s.email       = 'christian.orthmann@gmail.com'

data/lib/eeny-meeny/middleware.rb

@@ -24,34 +24,15 @@ module EenyMeeny
       query_parameters = query_hash(env)
       now              = Time.zone.now
       new_cookies      = {}
+      delete_cookies   = find_deprecated_cookies(cookies, now)
       # Prepare for experiments.
       @experiments.each do |experiment|
         # Skip inactive experiments
         next unless experiment.active?(now)
-        # Trigger experiment through query parameters
-        cookie_name = EenyMeeny::Cookie.cookie_name(experiment)
-        has_experiment_trigger = EenyMeeny.config.query_parameters[:experiment] && query_parameters.key?(cookie_name)
-        # skip experiments that already have a cookie
-        next unless has_experiment_trigger || !cookies.key?(cookie_name)
-        cookie = if has_experiment_trigger
-                   # Trigger experiment variation through query parameter.
-                   EenyMeeny::Cookie.create_for_experiment_variation(experiment, query_parameters[cookie_name].to_sym, @cookie_config)
-                 else
-                   EenyMeeny::Cookie.create_for_experiment(experiment, @cookie_config)
-                 end
-        # Set HTTP_COOKIE header to enable experiment on first pageview
-        env = add_or_replace_http_cookie(env, cookie)
-        new_cookies[cookie.name] = cookie
-      end
-      # Prepare smoke tests (if enabled through query parameters)
-      if EenyMeeny.config.query_parameters[:smoke_test]
-        if query_parameters.key?('smoke_test_id') && (query_parameters['smoke_test_id'] =~ /\A[A-Za-z_]+\z/)
-          # Set HTTP_COOKIE header to enable smoke test on first pageview
-          cookie = EenyMeeny::Cookie.create_for_smoke_test(query_parameters['smoke_test_id'], @cookie_config)
-          env = add_or_replace_http_cookie(env, cookie)
-          new_cookies[cookie.name] = cookie
-        end
+        env, new_cookies = prepare_experiment(env, cookies, new_cookies, query_parameters, experiment)
       end
+      # Prepare smoke tests
+      env, new_cookies = prepare_smoke_test(env, new_cookies, query_parameters)
       # Delegate to app
       status, headers, body = @app.call(env)
       response = Rack::Response.new(body, status, headers)
@@ -59,11 +40,57 @@ module EenyMeeny
       new_cookies.each do |key, value|
         response.set_cookie(key,value.to_h)
       end
+      delete_cookies.each do |key, value|
+        response.delete_cookie(key, value: value, path: @cookie_config[:path], same_site: @cookie_config[:same_site])
+      end
       response.finish
     end
 
     private
 
+    def find_deprecated_cookies(cookies, now)
+      deprecated_cookies = {}
+      cookies.each do |cookie_name, value|
+        # Skip any cookie that does not have the 'eeny_meeny_' prefix
+        next unless cookie_name.to_s.start_with?(EenyMeeny::Cookie::EXPERIMENT_PREFIX)
+        # Mark cookies that does not match any existing active experiment as 'deprecated'.
+        experiment = EenyMeeny::Experiment.find_by_cookie_name(cookie_name)
+        next if experiment && experiment.active?(now)
+        deprecated_cookies[cookie_name] = value
+      end
+      deprecated_cookies
+    end
+
+    def prepare_experiment(env, cookies, new_cookies, query_parameters, experiment)
+      # Trigger experiment through query parameters
+      cookie_name = EenyMeeny::Cookie.cookie_name(experiment)
+      has_experiment_trigger = EenyMeeny.config.query_parameters[:experiment] && query_parameters.key?(cookie_name)
+      # Skip experiments that already have a cookie
+      return env, new_cookies unless has_experiment_trigger || !cookies.key?(cookie_name)
+      cookie = if has_experiment_trigger
+                 # Trigger experiment variation through query parameter.
+                 EenyMeeny::Cookie.create_for_experiment_variation(experiment, query_parameters[cookie_name].to_sym, @cookie_config)
+               else
+                 EenyMeeny::Cookie.create_for_experiment(experiment, @cookie_config)
+               end
+      # Set HTTP_COOKIE header to enable experiment on first pageview
+      env = add_or_replace_http_cookie(env, cookie)
+      new_cookies[cookie.name] = cookie
+      return env, new_cookies
+    end
+
+    def prepare_smoke_test(env, new_cookies, query_parameters)
+      # Skip if the EenyMeeny 'smoke test' query parameters configuration is disabled.
+      return env, new_cookies unless EenyMeeny.config.query_parameters[:smoke_test]
+      # Skip if no valid 'smoke_test_id' query parameter present
+      return env, new_cookies unless query_parameters.key?('smoke_test_id') && (query_parameters['smoke_test_id'] =~ /\A[A-Za-z_]+\z/)
+      # Set HTTP_COOKIE header to enable smoke test on first pageview
+      cookie = EenyMeeny::Cookie.create_for_smoke_test(query_parameters['smoke_test_id'], @cookie_config)
+      env = add_or_replace_http_cookie(env, cookie)
+      new_cookies[cookie.name] = cookie
+      return env, new_cookies
+    end
+
     def query_hash(env)
       # Query Params are only relevant if EenyMeeny.config have them enabled.
       return {} unless EenyMeeny.config.query_parameters[:experiment] || EenyMeeny.config.query_parameters[:smoke_test]

data/lib/eeny-meeny/models/cookie.rb

@@ -2,7 +2,7 @@ require 'rack'
 
 module EenyMeeny
   class Cookie
-    COOKIE_PREFIX = 'eeny_meeny_'.freeze
+    EXPERIMENT_PREFIX = 'eeny_meeny_'.freeze
     SMOKE_TEST_PREFIX = 'smoke_test_'.freeze
 
     attr_accessor :value
@@ -52,7 +52,7 @@ module EenyMeeny
 
     def self.cookie_name(experiment)
       return if experiment.nil?
-      COOKIE_PREFIX+experiment.id.to_s+'_v'+experiment.version.to_s
+      EXPERIMENT_PREFIX+experiment.id.to_s+'_v'+experiment.version.to_s
     end
 
     def self.read(cookie_string)

data/lib/eeny-meeny/models/experiment.rb

@@ -4,6 +4,9 @@ require 'active_support/core_ext/enumerable'
 
 module EenyMeeny
   class Experiment
+
+    COOKIE_EXPERIMENT_ID_REGEX = /\Aeeny_meeny_(.+)_v\d+\z/.freeze
+
     attr_reader :id, :name, :version, :variations, :total_weight, :end_at, :start_at
 
     def self.find_all
@@ -17,6 +20,12 @@ module EenyMeeny
       new(experiment_id, **experiment) if experiment
     end
 
+    def self.find_by_cookie_name(cookie_name)
+      if cookie_name =~ COOKIE_EXPERIMENT_ID_REGEX
+        find_by_id($1)
+      end
+    end
+
     def initialize(id, name: '', version: 1, variations: {}, start_at: nil, end_at: nil)
       @id = id
       @name = name

data/lib/eeny-meeny/version.rb

@@ -1,3 +1,3 @@
 module EenyMeeny
-  VERSION = '2.1.4'
+  VERSION = '2.2.0'
 end

data/spec/eeny-meeny/middleware_spec.rb

@@ -70,6 +70,22 @@ describe EenyMeeny::Middleware do
       end
     end
 
+    context 'and the request contains a cookie from an undefined experiment' do
+      let(:request) { Rack::MockRequest.new(subject) }
+
+      before(:example) do
+        @original_request_cookies = 'test=abc;eeny_meeny_undefined_experiment_v1=thevaluedoesntmatter;'
+        @response = request.get('/test',
+                                'CONTENT_TYPE' => 'text/plain',
+                                'HTTP_COOKIE' => @original_request_cookies)
+      end
+
+
+      it "instructs the browser to remove through the 'Set-Cookie' header on the response" do
+        expect(@response['Set-Cookie']).to include('eeny_meeny_undefined_experiment_v1=thevaluedoesntmatter; path=/; max-age=0; expires=Thu, 01 Jan 1970 00:00:00 -0000; SameSite=Strict')
+      end
+    end
+
     context 'and given an experiment query parameter' do
       let(:request) { Rack::MockRequest.new(initialize_app(secure: false)) }
 

data/spec/eeny-meeny/models/experiment_spec.rb

@@ -194,4 +194,17 @@ describe EenyMeeny::Experiment do
     end
   end
 
+  describe '.find_by_cookie_name', experiments: true do
+    context 'when the given cookie name matches a configured experiment' do
+      it 'returns the experiment' do
+        expect(described_class.find_by_cookie_name(:eeny_meeny_my_page_v1)).to be_a EenyMeeny::Experiment
+      end
+    end
+
+    context 'when the given cookie name does not match a configured experiment' do
+      it 'returns nil' do
+        expect(described_class.find_by_cookie_name(:eeny_meeny_undefined_experiment_v2)).to be_nil
+      end
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: eeny-meeny
 version: !ruby/object:Gem::Version
-  version: 2.1.4
+  version: 2.2.0
 platform: ruby
 authors:
 - Christian Orthmann
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-10-04 00:00:00.000000000 Z
+date: 2018-07-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -190,9 +190,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.8
+rubygems_version: 2.6.13
 signing_key: 
 specification_version: 4
 summary: A simple split and smoke testing tool for Rails
 test_files: []
-has_rdoc: