Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Password challenge #40

Closed
rgvy opened this issue Jul 29, 2017 · 5 comments
Closed

Password challenge #40

rgvy opened this issue Jul 29, 2017 · 5 comments

Comments

@rgvy
Copy link

rgvy commented Jul 29, 2017

Hello

Just a quick feature suggestion: the ability to challenge a user for a password without proceeding through a complete login process.

For example, imagine that a user is already authenticated because they are remembered. But now the user accesses a "dangerous" or "significant" feature (such as linking a bank account). It would be good to challenge the user for their password again to ensure that it is not an imposter using a shared computer, for instance.

Browsing the internals of Auth.php I see that this can be done by password_verify($password, $userData['password']). However, perhaps there is some more reusable logic that may be helpful. Or if the password hashing changes, it would be good to encapsulate it all in one place...

Thx

@ocram
Copy link
Contributor

ocram commented Aug 5, 2017

Thank you very much!

You're right about everything, and this is definitely needed. That feature has indeed been envisaged from the beginning, which is why we have isRemembered in the first place. But somehow it was forgotten later on.

Will be available shortly!

@p08dev
Copy link

p08dev commented Aug 17, 2017

I'm very interested in this feature! Any information on when this will be available?

@ocram
Copy link
Contributor

ocram commented Aug 18, 2017

@Furentes Within the next week!

@p08dev
Copy link

p08dev commented Aug 19, 2017

Awesome! 👍

@ocram
Copy link
Contributor

ocram commented Aug 25, 2017

Implemented in:

This is available in a new major version, v6.0.0, which comes with a few breaking changes. For a guide on how to upgrade, please see the migration notes in Migration.md.

@ocram ocram closed this as completed Aug 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants