Comments on: open_id_authentication (part 2) http://leancode.com/2007/04/04/open_id_authentication-part-2/ bernie's small batches of software goodness Fri, 21 Nov 2008 12:42:03 +0000 http://wordpress.org/?v=2.6.3 By: Yoshi http://leancode.com/2007/04/04/open_id_authentication-part-2/#comment-24411 Yoshi Sat, 29 Dec 2007 10:06:50 +0000 http://leancode.com/2007/04/04/open_id_authentication-part-2/#comment-24411 Thanks. this informations helps me a lot. by the way, "AuthenticatedSessionController" should be "AuthenticatedSession[s]Controller" and "@session[:user] = nil" in the destroy method should be "session[:user_id] = nil" I think. any way, thanks a lot. Thanks. this informations helps me a lot.
by the way,
“AuthenticatedSessionController” should be “AuthenticatedSession[s]Controller” and “@session[:user] = nil” in the destroy method should be “session[:user_id] = nil” I think.
any way, thanks a lot.

]]> By: Bernie Thompson http://leancode.com/2007/04/04/open_id_authentication-part-2/#comment-14820 Bernie Thompson Sat, 28 Jul 2007 23:35:16 +0000 http://leancode.com/2007/04/04/open_id_authentication-part-2/#comment-14820 Hi Ryan, You're right, there appears to be an error in persisting the remember_me param from begin() through the redirects to complete(). In part one of the article (changes to DHH's lib), there should be additional changes to add the remember_me param to the redirect URL (line 115, as I have the line numbers in the diffs in part 1 of the article). To make that happen, you'll also have to add paramters to the routines there to pass down whether remember_me is set. Thanks for catching this error -- let us know if you code up a solution that works! Hi Ryan,

You’re right, there appears to be an error in persisting the remember_me param from begin() through the redirects to complete(). In part one of the article (changes to DHH’s lib), there should be additional changes to add the remember_me param to the redirect URL (line 115, as I have the line numbers in the diffs in part 1 of the article). To make that happen, you’ll also have to add paramters to the routines there to pass down whether remember_me is set.

Thanks for catching this error — let us know if you code up a solution that works!

]]> By: ryan http://leancode.com/2007/04/04/open_id_authentication-part-2/#comment-14762 ryan Sat, 28 Jul 2007 03:25:06 +0000 http://leancode.com/2007/04/04/open_id_authentication-part-2/#comment-14762 I'm trying to get the :remember_me functionality to work with open_id_auth. Since it leaves to verify at the identity server, the remember_me param is wiped out when it gets back. What are your thoughts? I’m trying to get the :remember_me functionality to work with open_id_auth. Since it leaves to verify at the identity server, the remember_me param is wiped out when it gets back. What are your thoughts?

]]>