Skip to content

Commit 2770c41

Browse files
Moves redis session management to own class
1 parent 0fd428e commit 2770c41

File tree

4 files changed

+4
-3
lines changed

4 files changed

+4
-3
lines changed

app/controllers/kracken/sessions_controller.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ def create
88
@user = user_class.find_or_create_from_auth_hash(auth_hash)
99
session[:user_id] = @user.id
1010
session[:user_uid] = @user.uid
11-
session[:user_cache_key] = SessionManager.get(@user.uid)
11+
session[:user_cache_key] = ::SessionManager.get(@user.uid)
1212
session[:token_expires_at] = Time.zone.at(auth_hash[:credentials][:expires_at])
1313

1414
redirect_to return_to_path

lib/kracken.rb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
require "kracken/authenticator"
1313
require "kracken/registration"
1414
require "kracken/railtie" if defined?(Rails)
15+
require "kracken/session_manager"
1516

1617
module Kracken
1718
mattr_accessor :config

lib/kracken/controllers/authenticatable.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,7 @@ def session_present?
117117
end
118118

119119
def session_and_redis_match?
120-
SessionManager.update(session[:user_uid]) == session[:user_cache_key]
120+
::SessionManager.get(session[:user_uid]) == session[:user_cache_key]
121121
end
122122

123123
def delete_session_data

lib/kracken/session_manager.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ def self.get(user_id)
1313
conn.get(user_session_key(user_id))
1414
end
1515

16-
def self.clear(user_id)
16+
def self.del(user_id)
1717
return unless active?
1818

1919
conn.del(user_session_key(user_id))

0 commit comments

Comments
 (0)