remove NullSession

lib/rack/session/abstract/id.rb

@@ -14,11 +14,6 @@ module Rack
 
   module Session
 
-    class NullSessionId
-      def empty?; true; end
-      def nil?; true; end
-    end
-
     class SessionId
       attr_reader :public_id
 
@@ -299,7 +294,7 @@ def prepare_session(env)
         def load_session(env)
           sid = current_session_id(env)
           sid, session = get_session(env, sid)
-          [sid || NullSessionId.new, session || {}]
+          [sid, session || {}]
         end
 
         # Extract session id from request object.
@@ -308,7 +303,7 @@ def extract_session_id(env)
           request = Rack::Request.new(env)
           sid = request.cookies[@key]
           sid ||= request.params[@key] unless @cookie_only
-          (sid && SessionId.new(sid)) || NullSessionId.new
+          sid && SessionId.new(sid)
         end
 
         # Returns the current session id from the SessionHash.

lib/rack/session/cookie.rb

@@ -126,7 +126,7 @@ def get_session(env, sid)
       end
 
       def extract_session_id(env)
-        unpacked_cookie_data(env)["session_id"] || NullSessionId.new
+        unpacked_cookie_data(env)["session_id"]
       end
 
       def unpacked_cookie_data(env)
@@ -178,11 +178,7 @@ def set_session(env, session_id, session, options)
 
       def destroy_session(env, session_id, options)
         # Nothing to do here, data is in the client
-        if options[:drop]
-          NullSessionId.new
-        else
-          generate_sid
-        end
+        generate_sid unless options[:drop]
       end
 
       def digest_match?(data, digest)

lib/rack/session/pool.rb

@@ -61,11 +61,7 @@ def set_session(env, session_id, new_session, options)
       def destroy_session(env, session_id, options)
         with_lock(env) do
           @pool.delete(session_id.private_id)
-          if options[:drop]
-            NullSessionId.new
-          else
-            generate_sid
-          end
+          generate_sid unless options[:drop]
         end
       end