Add a version prefix to the private id to make easier to migrate old values

rafaelfranca · tenderlove · commit 5b1cab667270 · 2019-12-17T09:54:34.000-08:00
diff --git a/lib/rack/session/abstract/id.rb b/lib/rack/session/abstract/id.rb
@@ -13,14 +13,16 @@ module Rack
   module Session
 
     class SessionId
+      ID_VERSION = 2
+
       attr_reader :public_id
 
       def initialize(public_id)
         @public_id = public_id
       end
 
       def private_id
-        hash_sid public_id
+        "#{ID_VERSION}::#{hash_sid(public_id)}"
       end
 
       alias :cookie_value :public_id
diff --git a/test/spec_session_memcache.rb b/test/spec_session_memcache.rb
@@ -246,7 +246,6 @@
       pool.pool.set(session_id.public_id, ses0, 0, true)
       pool.pool.delete(session_id.private_id)
 
-
       res1 = req.get("/", "HTTP_COOKIE" => cookie)
       res1["Set-Cookie"].must_be_nil
       res1.body.must_equal '{"counter"=>2}'
@@ -266,7 +265,6 @@
       pool.pool.set(session_id.public_id, ses0, 0, true)
       pool.pool.delete(session_id.private_id)
 
-
       res2 = dreq.get("/", "HTTP_COOKIE" => cookie)
       res2["Set-Cookie"].must_be_nil
       res2.body.must_equal '{"counter"=>2}'
