Skip to content

Commit 80254e8

Browse files
Look up cache key info from user UID, not ID
1 parent 4f6a019 commit 80254e8

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

app/controllers/kracken/sessions_controller.rb

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,10 @@ class SessionsController < ActionController::Base
77
def create
88
@user = user_class.find_or_create_from_auth_hash(auth_hash)
99
session[:user_id] = @user.id
10-
session[:user_cache_key] = SESSION_REDIS.get(user_session_key(@user.id))
10+
session[:user_uid] = @user.uid
11+
session[:user_cache_key] = SESSION_REDIS.get(user_session_key(@user.uid))
1112
session[:token_expires_at] = Time.zone.at(auth_hash[:credentials][:expires_at])
13+
1214
redirect_to return_to_path
1315
end
1416

lib/kracken/controllers/authenticatable.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ def session_present?
129129
end
130130

131131
def session_and_redis_match?
132-
SESSION_REDIS.get(user_session_key(session[:user_id])) == session[:user_cache_key]
132+
SESSION_REDIS.get(user_session_key(session[:user_uid])) == session[:user_cache_key]
133133
end
134134

135135
def user_session_key(id)

0 commit comments

Comments
 (0)