#83 ✓resolved

valid_password? leaves "transition" users invalid

Reported by Si | April 9th, 2009 @ 10:17 AM

When using valid_password? for a user that needs transition to a newer encryption, the update works as expected, but a subsequent call to valid? will incorrectly show the record is not valid. As per errors.full_messages:

Password confirmation is too short (minimum is 4 characters)

Setting the password_confirmation as well as the password inside valid_password? is a quick fix that works.

Comments and changes to this ticket

  • Ben Johnson

    Ben Johnson April 9th, 2009 @ 12:26 PM

    • State changed from “new” to “resolved”

    Good find, actually that is my fault. I just need to reset the @password_changed instance variable. I updated the repo, this should solve the problem.

  • Si

    Si April 9th, 2009 @ 12:44 PM

    Looks good, thanks.

  • merlin pendragon

    merlin pendragon February 14th, 2021 @ 05:38 AM

    The problem with this is that most people can tell when you are writing something just to write. You can’t always write about things that you believe strongly in, but it definitely helps. When writing for other people, there’s not really a way to get around writing about things that you don’t care about www.prepaidgiftbalance.com

  • 123VEGA

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

Object based authentication solution that handles all of the non sense for you. It's as easy as ActiveRecord is with a database.

People watching this ticket