TEST_CODEOWNERS/rails
3
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Fix argument naming in AbstractStore#set_cookie

Browse Source 
The signature in Rack https://github.com/rack/rack/blob/master/lib/rack/session/abstract/id.rb#L415 expects a response as the second argument
This commit is contained in:
Marc Rohloff 2020-07-24 19:07:14 -06:00 committed by GitHub
parent 4fe1452534
commit 9cc8e51e0d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
actionpack/lib/action_dispatch/middleware/session/abstract_store.rb
View File

@ -82,7 +82,7 @@ class AbstractStore < Rack::Session::Abstract::Persisted
include SessionObject
private
def set_cookie(request, session_id, cookie)
def set_cookie(request, response, cookie)
request.cookie_jar[key] = cookie
end
end
@ -97,7 +97,7 @@ def generate_sid
end
private
def set_cookie(request, session_id, cookie)
def set_cookie(request, response, cookie)
request.cookie_jar[key] = cookie
end
end