Added Remember Me feature

[bahymohammmed]

This is a new feature to add remember me feature in login view.

Here's the change log:

• In class Authenticator I've changed toCookie method signature to accept a new argument which is rememberCookie as boolean
• I've rewrote the structure of the login form to accept the LoginInfo object instead of accepting tuple(String, String)
• Add a new class case class LoginInfo(username: String, password: String, remember: Option[Boolean]) to be used in the new form loginFormWithoutRemember instead of accepting two strings as username and password, so we pass the remember me option along side with the credentials
• Added lazy val withRememberMe = current.configuration.getBoolean(RememberMe).getOrElse(false)
• Modify the view to add remember me checkbox at /views/provider.scala.html

[tuxBurner]

Coool one :)

[bahymohammmed]

Thanks :-)

[vchuravy]

This would be quite nice to have :) Is there anything holding it back from being merged? I would be willing to do some work if it needs to be polished up.

[bahymohammmed]

really don't know why it's not merged yet !

Thanks for your support :-)

[vchuravy]

@jaliss Any comments? I really like your work on securesocial, but this is a bit that is missing for me.

[Ingwar]

+1

[arpadtamasi]

+1

[jaliss]

@bahymohammmed is this working for you? From what I see you are making the cookie transient if the user checks on the "remember me" option at login time.

[AdrianScott]

This would really help speed up dev time, so one doesn't have to re-login after each server reload...

[jaliss]

@AdrianScott if you want to prevent loosing your session on reach reload you can write your own version of the AuthenticatorStore (that does not use the Play cache) or configure the Play cache so that it persists things in disk. To do that change the ehcache.xml file and set diskPersistent="true".

[AdrianScott]

Thank you! I appreciate it. (still needing a 'remember me' for my users sometime soon also, hehe )

[jaliss]

@AdrianScott you're welcome. Will keep this in mind :)

[ludwigm]

+1 (Any news on integrating this? I think this is a very important feature I would really appreciate)

[chazzzz]

Hi! I'm currently needing this feature. Any news?

[enotys]

Why not merged this feature ? I need It! It is very important feature!