Better nil guards from calling cookie value on nil or string

[Fixes charlotte-ruby#292]
rposborne · Jan 19, 2021 · 6a01657 · 6a01657
1 parent 46a582f
commit 6a01657
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/app/controllers/impressionist_controller.rb b/app/controllers/impressionist_controller.rb
@@ -154,7 +154,7 @@ def session_hash
         id = request.session_options[:id]
       end
 
-      unless id.is_a? String
+      if id.respond_to?(:cookie_value)
         id = id.cookie_value if Rack::Session::SessionId.const_defined?(:ID_VERSION) && Rack::Session::SessionId::ID_VERSION == 2
       end