[LRUG] Notes from my talk on refactoring
John M Forsyth
johnmforsyth at yahoo.co.uk
Thu Feb 23 09:07:18 PST 2012
Just wanted to let you know that I thought it was a really good talk: very concrete and you did a great job of explaining thought processes in very limited time!
From: Richard Livsey <richard at livsey.org>
To: ikennaokpala at googlemail.com; London Ruby Users Group <chat at lists.lrug.org>
Sent: Thursday, 23 February 2012, 16:47
Subject: Re: [LRUG] Notes from my talk on refactoring
Yeah, authenticate_with_password would probably take a username and password if done in reality, passing round the params hash isn't very nice. In reality if using OmniAuth/similar you'd probably pass the auth_hash through instead of params.
In the end it's not really about authentication more a case of illustrating the refactoring side of things, so I admit I may have 'cheated' a bit by knowing the outcome when I started off writing the first example :o)
Glad you enjoyed it!
On Thursday, 23 February 2012 at 16:34, Okpala Ikenna N. wrote:
> Jus finished reading your post.. to put long short.. "Brilliant".
> i am wondering why you have choosen to allow authenticate_with_password(params) to be aware of opening up the params hash.
> On Thu, Feb 23, 2012 at 4:03 PM, Richard Livsey <richard at livsey.org (mailto:richard at livsey.org)> wrote:
> > I've written up my notes for the lightning talk I gave on refactoring:
> > http://livsey.org/blog/2012/02/23/should-your-user-care-about-authentication/
> > Hopefully it is of interest to someone!
> > Cheers.
> > --
> > Richard Livsey
> > Co-Founder, MinuteBase
> > Meeting collaboration made easy
> > http://minutebase.com
> > +44 (0) 7841 260 797 (tel:%2B44%20%280%29%207841%20260%20797)
> > _______________________________________________
> > Chat mailing list
> > Chat at lists.lrug.org (mailto:Chat at lists.lrug.org)
> > http://lists.lrug.org/listinfo.cgi/chat-lrug.org
> Chat mailing list
> Chat at lists.lrug.org (mailto:Chat at lists.lrug.org)
Chat mailing list
Chat at lists.lrug.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the Chat