Skip to content
This repository has been archived by the owner on Jul 21, 2020. It is now read-only.

remember_me will never work? #5

Open
jfqd opened this issue Nov 5, 2009 · 1 comment
Open

remember_me will never work? #5

jfqd opened this issue Nov 5, 2009 · 1 comment

Comments

@jfqd
Copy link

jfqd commented Nov 5, 2009

Am I wrong or will remember_me never work? In my opinion two lines are missing in 'by_cookie_token.rb' file (see patch):

Index: restful_authentication/lib/authentication/by_cookie_token.rb

--- restful_authentication/lib/authentication/by_cookie_token.rb
+++ restful_authentication/lib/authentication/by_cookie_token.rb
@@ -42,8 +42,10 @@
# refresh token (keeping same expires_at) if it exists
def refresh_token
if remember_token?

  •      self.remember_token = self.class.make_token 
    
  •      save(false)  
    
  •      self.remember_token = self.class.make_token
    
  •      save(false)
    
  •    else
    
  •      remember_me
     end
    
    end
@technoweenie
Copy link
Owner

file a busted test that proves what you're trying to say, or even file a fix with it :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants